This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Fix PR target/19933 and PR target/21315


On Thu, 12 May 2005, Eric Botcazou wrote:

> fixincludes/
> 	* inclhack.def: New fixes solaris_math_[1-10].
> 	* fixincl.x: Regenerate.
> 	* tests/base/iso/math_c99.h: New.

NAN should expand to use __builtin_nanf not __builtin_nan, as its value is 
meant to be of type float.

The bypasses look overly specific to me (presuming the precise form of a 
future header will mention __GNUC__ on the line before a particular 
definition).  As they are all for the benefit of a hypothetical future 
header, I'd rather they just all bypassed on __GNUC__ and then if a future 
Solaris version does get released with some but not all cases fixed then 
the testcases you add should detect this and the bypasses can be refined 
then.

-- 
Joseph S. Myers               http://www.srcf.ucam.org/~jsm28/gcc/
    jsm@polyomino.org.uk (personal mail)
    joseph@codesourcery.com (CodeSourcery mail)
    jsm28@gcc.gnu.org (Bugzilla assignments and CCs)


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]