Fix arm-eabi attributes

Richard Earnshaw rearnsha@arm.com
Thu Apr 30 16:24:00 GMT 2009


On Thu, 2009-04-30 at 15:24 +0100, Nathan Sidwell wrote:
> This patch fixes a latent problem with arm eabi attributes.  We triggered off 
> TE_LINUX when defaulting the FPU, thus leading to a lack of default for plain 
> arm-eabi, leading to incorrect object files.
> 

> ! #if !(defined (TE_LINUX) || defined (TE_NetBSD) || defined (TE_VXWORKS))
>         /* Some environments specify a default FPU.  If they don't, infer it
>   	 from the processor.  */
>         if (mcpu_fpu_opt)
> --- 20267,20273 ----
>       }
>     else if (!mfpu_opt)
>       {
> ! #if !(defined (EABI_DEFAULT) || defined (TE_NetBSD) || defined (TE_VXWORKS))

I'm not sure that's right for legacy linux (which defaults to FPA);
shouldn't you just add EABI_DEFAULT to the list rather than replace
TE_LINUX with it?

R.



More information about the Binutils mailing list