Re: [i386, patch, RFC] HLE support in GCC

[Jakub Jelinek <jakub at redhat dot com>]

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