On Wed, Apr 11, 2012 at 06:06:58PM +0200, Andi Kleen wrote: > +static unsigned HOST_WIDE_INT > +ix86_extend_hle_macro (unsigned HOST_WIDE_INT memmodel) > +{ > + unsigned HOST_WIDE_INT result = memmodel; > + > + if (memmodel & IX86_HLE_ACQUIRE) > + result |= MEMMODEL_ACQUIRE; > + > + if (memmodel & IX86_HLE_RELEASE) > + result |= MEMMODEL_RELEASE; > + > + return result; > +}
I actually think it is a bad idea to imply any memory model from the HLE bits. If anything, we should warn for memmodel + hle bit combinations that are unlikely to DTRT. Let the developers really use the model they want. Jakub