On Wed, Apr 11, 2012 at 03:06:35PM +0200, Andi Kleen wrote:
> Do you really imply ACQUIRE/RELEASE with HLE_ACQUIRE/RELEASE now? I don't
> see that in the code. I think that's really required, otherwise the optimizer
> will do the wrong thing and move memory references outside the region.
IMHO the separate bits for HLE_ACQUIRE/RELEASE are desirable, you could very
well use __ATOMIC_ACQ_REL or __ATOMIC_SEQ_CST if you want stronger memory
barrier, still coupled with either __ATOMIC_HLE_ACQUIRE or
__ATOMIC_HLE_RELEASE...
Jakub
