[cfe-commits] r110096 - in /cfe/trunk: include/clang/Basic/BuiltinsX86.def test/CodeGen/builtins-x86.c

[Chris Lattner]

On Aug 2, 2010, at 9:10 PM, Bruno Cardoso Lopes wrote:

> Hi Chris,
> 
>> Hey Bruno,
>> 
>> Is __builtin_ia32_addpd256 just a simple add?  We've preferred to not add builtins for operations that can be trivially matched from LLVM IR.  The Clang header for AVX should just define the Xmmintrin.h functions in terms of "+" on extended vectors (for example).  The other headers in clang/lib/Headers are implemented the same way.
>> 
>> We still need to have builtins for really crazy stuff of course.
> 
> That's the idea, and what I tried to explain in the commit message.

Ok, I saw and responded to the clang patch before I saw the llvm patch, thanks!

-Chris