From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <amonakov@ispras.ru>
Received: from mail.ispras.ru (mail.ispras.ru [83.149.199.84])
	by sourceware.org (Postfix) with ESMTPS id 2F2493955C82
	for <gcc-patches@gcc.gnu.org>; Wed, 16 Nov 2022 07:51:48 +0000 (GMT)
DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 2F2493955C82
Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=ispras.ru
Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=ispras.ru
Received: from [10.10.3.121] (unknown [10.10.3.121])
	by mail.ispras.ru (Postfix) with ESMTPS id 71B0C419E9CE;
	Wed, 16 Nov 2022 07:51:42 +0000 (UTC)
DKIM-Filter: OpenDKIM Filter v2.11.0 mail.ispras.ru 71B0C419E9CE
Date: Wed, 16 Nov 2022 10:51:42 +0300 (MSK)
From: Alexander Monakov <amonakov@ispras.ru>
To: Hongyu Wang <wwwhhhyyy333@gmail.com>
cc: Jonathan Wakely <jwakely@redhat.com>, Hongyu Wang <hongyu.wang@intel.com>, 
    gcc-patches@gcc.gnu.org, hongtao.liu@intel.com, ubizjak@gmail.com
Subject: Re: [PATCH] doc: Reword the description of -mrelax-cmpxchg-loop [PR
 107676]
In-Reply-To: <CA+OydW=gyAZN8C42nMxxK7V0U8xRtDxADehsYSLSNhVxoC3fLQ@mail.gmail.com>
Message-ID: <36b75ee4-5f97-16ab-8d9d-a060e996b5da@ispras.ru>
References: <20221115033559.66827-1-hongyu.wang@intel.com> <Y3Nvq7zZT/JRhAtX@redhat.com> <9289c261-6aeb-2fdf-6599-4e8d77c30f8@ispras.ru> <CACb0b4mxggf4jPuKGHL=PAjoFnWgm0efjYYaW3S6UjX2-XAP0Q@mail.gmail.com> <7a41f182-1638-1a70-c0dc-b90b1985c31@ispras.ru>
 <CA+OydW=gyAZN8C42nMxxK7V0U8xRtDxADehsYSLSNhVxoC3fLQ@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
X-Spam-Status: No, score=-3.1 required=5.0 tests=BAYES_00,KAM_DMARC_STATUS,SPF_HELO_NONE,SPF_PASS,TXREP autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org
List-Id: <gcc-patches.gcc.gnu.org>


On Wed, 16 Nov 2022, Hongyu Wang wrote:

> > When emitting a compare-and-swap loop for @ref{__sync Builtins}
> > and @ref{__atomic Builtins} lacking a native instruction, optimize
> > for the highly contended case by issuing an atomic load before the
> > @code{CMPXCHG} instruction, and using the @code{PAUSE} instruction
> > to save CPU power when restarting the loop.
> 
> Thanks for the correction, it looks quite clear now! Here is the
> updated patch, ok for trunk?

Please use 'git commit --author' to indicate authorship of the patch
(or simply let me push it once approved).

Jonathan, please let us know if the above wording looks fine to you?
Mainly I'm asking if '... and using' or '... and use' is easier to read.

Alexander