newlib ieeefp.h again

[Andrew Johnson]

On 03/23/2017 01:58 AM, Sebastian Huber wrote:
> On 23/03/17 00:17, Andrew Johnson wrote:
>> Newlib's setting for _LDBL_EQ_DBL (in newlib.h) needs to change based on
>> the particular CPU being compiled for. Here's some evidence:
> 
> Yes, this definition of _LDBL_EQ_DBL is wrong, since it depends on the
> multilib. For example we have in the GCC build tree:
> 
> grep _LDBL_EQ_DBL `find -name newlib.h`
> ./m68k-rtems4.12/m68040/softfp/newlib/newlib.h:/* #undef _LDBL_EQ_DBL */
> ./m68k-rtems4.12/m68040/newlib/newlib.h:/* #undef _LDBL_EQ_DBL */
> ./m68k-rtems4.12/m5208/newlib/newlib.h:#define _LDBL_EQ_DBL 1
> ./m68k-rtems4.12/m5329/newlib/newlib.h:#define _LDBL_EQ_DBL 1
> ./m68k-rtems4.12/m5206/newlib/newlib.h:#define _LDBL_EQ_DBL 1
> ./m68k-rtems4.12/m5407/newlib/newlib.h:#define _LDBL_EQ_DBL 1
> ./m68k-rtems4.12/softfp/newlib/newlib.h:/* #undef _LDBL_EQ_DBL */
> ./m68k-rtems4.12/m68000/newlib/targ-include/newlib.h:/* #undef _LDBL_EQ_DBL */
> ./m68k-rtems4.12/m68000/newlib/newlib.h:/* #undef _LDBL_EQ_DBL */
> ./m68k-rtems4.12/newlib/targ-include/newlib.h:/* #undef _LDBL_EQ_DBL */
> ./m68k-rtems4.12/newlib/newlib.h:/* #undef _LDBL_EQ_DBL */
> ./m68k-rtems4.12/m5475/softfp/newlib/newlib.h:#define _LDBL_EQ_DBL 1
> ./m68k-rtems4.12/m5475/newlib/newlib.h:#define _LDBL_EQ_DBL 1
> ./m68k-rtems4.12/m68060/softfp/newlib/newlib.h:/* #undef _LDBL_EQ_DBL */
> ./m68k-rtems4.12/m68060/newlib/newlib.h:/* #undef _LDBL_EQ_DBL */
> ./m68k-rtems4.12/mcpu32/newlib/newlib.h:/* #undef _LDBL_EQ_DBL */
> ./m68k-rtems4.12/m5307/newlib/newlib.h:#define _LDBL_EQ_DBL 1

Are there any other settings in those newlib.h files that change
depending on the multilib? I would suggest checking the other CPU
families too, better to fix any similar issues all at once...

- Andrew

-- 
Arguing for surveillance because you have nothing to hide is no
different than making the claim, "I don't care about freedom of
speech because I have nothing to say." -- Edward Snowdon