[PATCH] R600: Add intrinsics for math helper instructions

Matt Arsenault arsenm2 at gmail.com
Sun Jun 22 16:17:16 PDT 2014


On Jun 17, 2014, at 7:10 AM, Tom Stellard <tom at stellard.net> wrote:

> I think we should replace the r600 in the builtin name with amdgpu, this will
> prevent some confusion about what hardware is supported on.

I found a problem with this. getIntrinsicForGCCBuiltin / Clang uses Triple::getArchTypePrefix(), which is “r600” when looking for the GCCBuiltin. It never finds the ones with AMDGPU as the target prefix, so to support both of these prefixes clang would need to specially handle these to support both __builtin prefixes
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140622/2d165d13/attachment.html>


More information about the llvm-commits mailing list