[llvm-commits] CVS: llvm/include/llvm/Support/DataTypes.h.in

[Chris Lattner]

On Nov 8, 2006, at 9:50 AM, Jim Laskey wrote:
> We know (float)HUGE_VAL works because that's where we started.

Right, but if DataTypes.in.h doesn't #include <cmath> before checking  
for HUGE_VALF, HUGE_VALF will never be defined.

-Chris

> -- Jim
>
>
> On Nov 8, 2006, at 1:41 PM, Chris Lattner wrote:
>
>>>
>>> #ifndef HUGE_VALF
>>> #ifndef HUGE_VAL
>>> #ifndef MAX_FLT
>>> #warning don't know how to define HUGE_VALF
>>> #else
>>> #define HUGE_VALF MAX_FLT
>>> #endif
>>> #else
>>> #define HUGE_VALF (float)HUGE_VAL
>>> #endif
>>> #endif
>>
>> More significantly, neither of these will work unless you #include  
>> <cmath> and/or float.h.
>>
>> -Chris
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits