[llvm-commits] [llvm] r150060 - in /llvm/trunk: include/llvm/IntrinsicsX86.td lib/Target/X86/X86ISelLowering.cpp lib/Target/X86/X86InstrSSE.td

[David A. Greene]

Craig Topper <craig.topper at gmail.com> writes:

> I do intend to remove the intrinsics eventually. I just haven't looked
> at what I need to do in order to do that in the AutoUpgrade code so I
> didn't do it yet. The previous intrinsic removal was my first time
> using the IR builder functions so I'm pretty new to that side of llvm.
>
> Would you prefer me to put the builtin lines back or would you rather
> fix dragonegg to not use them?

Please put back the builtin lines and do not delete the intrinsics.

Intrinsics are used by users and we want to respect that.

The builtins are useful for running the C backend to get something
compilable by gcc.  This is very helpful for debugging.

                               -Dave