[llvm-commits] [llvm] r90199 - /llvm/trunk/include/llvm/IntrinsicsX86.td

[Eric Christopher]

On Dec 1, 2009, at 12:50 AM, Duncan Sands wrote:

> Hi Eric,
> 
>> Remove the gcc builtins from the intrinsics, we'll lower them
>> explicitly so we can check arguments.
> 
> I think the gcc builtin is also used by the C backend to go from the
> LLVM intrinsic to the gcc intrinsic.  Instead of doing this, how about
> flipping the logic in EmitBuiltinCall so it only checks
> getIntrinsicForGCCBuiltin if EmitFrontendExpandedBuiltinCall fails,
> rather than the other way round like now.

Seems like a rather good plan. I'll need to do that for clang as well. I'll get to it in the next day or so.

-eric