On Wed, May 25, 2011 at 3:31 AM, Richard Guenther
<richard.guent...@gmail.com> wrote:
> On Wed, May 25, 2011 at 12:26 PM, Andrew Haley <a...@redhat.com> wrote:
>> On 05/24/2011 07:28 PM, H.J. Lu wrote:
>>
>>> This patch implements pause intrinsic suggested by Andi. OK
>>> for trunk?
>>
>> What does "full memory barrier" here mean?
>>
>> +@table @code
>> +@item void __builtin_ia32_pause (void)
>> +Generates the @code{pause} machine instruction with full memory barrier.
>> +@end table
>>
>> There a memory clobber, but no barrier instruction AFAICS. The
>> doc needs to explain it a bit better.
>
> The name also sounds odd to me (reminds me of Fortran PAUSE ...).
>
__builtin_ia32_pause is the C intrinsic for x86 machine instruction.
I don't think people will get confused by its name.
--
H.J.
