patch: Optionally use -mlong-double-128 by default on sparc

Jakub Jelinek jakub@redhat.com
Fri Feb 3 22:08:00 GMT 2006


On Fri, Feb 03, 2006 at 11:04:49PM +0100, Eric Botcazou wrote:
> > How does this look?
> 
> Mostly OK I think, but...
> 
> > +#ifdef TARGET_DEFAULT_LONG_DOUBLE_128
> > +  if (target_flags_explicit & MASK_LONG_DOUBLE_128)
> > +    target_flags |= MASK_LONG_DOUBLE_128;
> > +#endif
> 
> ... has changed since the first version (disregarding !TARGET_64BIT):
> 
> +#if defined SPARC_LINUX && defined TARGET_DEFAULT_LONG_DOUBLE_128
> +  if (!TARGET_64BIT && !(target_flags_explicit & MASK_LONG_DOUBLE_128))
> +    target_flags |= MASK_LONG_DOUBLE_128;
> +#endif
> 
> My understanding is that the first version is the correct one, but I'll let 
> Jakub and you settle that.  I also think that Andrew's suggestion is sound.

Should be:

#ifdef TARGET_DEFAULT_LONG_DOUBLE_128
  if (! (target_flags_explicit & MASK_LONG_DOUBLE_128))
    target_flags |= MASK_LONG_DOUBLE_128;
#endif

(the topmost above is missing a negation).

	Jakub



More information about the Gcc-patches mailing list