[cfe-commits] r52914 - in /cfe/trunk: include/clang/AST/ASTContext.h include/clang/AST/Builtins.def include/clang/Basic/TargetInfo.h lib/AST/ASTContext.cpp lib/CodeGen/CGBuiltin.cpp lib/CodeGen/CGExprScalar.cpp lib/CodeGen/CodeGenTypes.cpp lib/Co

Chris Lattner clattner at apple.com
Mon Jun 30 22:30:23 PDT 2008


On Jun 30, 2008, at 10:15 PM, Eli Friedman wrote:

>> Because people do things like this: *shudder*
>>
>> double x = __builtin_nan("");
>>
>> __builtin_nan comes from system headers sometimes.  I will  
>> immediately and
>> completely agree that your new constant evaluation code is the  
>> right place
>> to handle this!
>
> Right... but it's worth noting that codegen for constant expressions
> never calls into CGBuiltin, so doing folding there doesn't help any.

Fair enough, hopefully that code can be moved to the right place when  
the generalized constant folding code comes online.

-Chris



More information about the cfe-commits mailing list