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

[Chris Lattner]

On Jan 7, 2010, at 12:45 PM, Eric Christopher wrote:

>
> On Jan 7, 2010, at 12:22 PM, David Greene wrote:
>
>> What llvm intrinsics?
>>
>> We don't "need" them in the sense of functionality, but gcc defines  
>> them
>> so it makes sense to support them.
>>
>> Adding them also makes the SIMD reorganization a bit simpler.
>
> I don't think you're understanding what Evan was saying.  We don't  
> need, at least, the "gcc intrinsic" part of it because the gcc  
> intrinsic should be lowered to explicit llvm instructions instead of  
> llvm intrinsics.
>
> The gcc intrinsics should be supported, but if possible they should  
> be lowered to instructions instead of intrinsics.

right.  David, please revert this patch.

-Chris