[llvm-commits] [llvm-gcc-4.2] r55783 - in /llvm-gcc-4.2/trunk/gcc: llvm-convert.cpp llvm-internal.h

[Dale Johannesen]

On Sep 4, 2008, at 12:39 PMPDT, Duncan Sands wrote:

> Hi Dale, since the llvm intrinsics are presumably readnone
> (hence don't set errno, and don't read the floating point
> mode - so they are what you want in "fast math" mode) shouldn't
> these expansions only be done if the gcc builtins are marked
> "const"?  IIRC, gcc marks these builtins "const" or "readonly"
> depending on platform requirements and whether fast-math is
> being used.

You may be right.  The way sqrt is handled maybe?

> Ciao,
>
> Duncan.
>
>> +  case BUILT_IN_POW:
>> +  case BUILT_IN_POWF:
>> +  case BUILT_IN_POWL:
>> +    Result = EmitBuiltinPOW(exp);
>> +    return true;
> ...