On Wed, May 25, 2011 at 3:31 AM, Richard Guenther <[email protected]> wrote: > On Wed, May 25, 2011 at 12:26 PM, Andrew Haley <[email protected]> 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.
