Missing TargetPrefix for NVVM intrinsics

[Justin Bogner via llvm-commits]

Justin Lebar <jlebar at google.com> writes:
>> I'll wait on the ones that drop the ptx intrinsics and builtins
>> until I hear back about the ones that don't seem to have an nvvm
>> equivalent.
>
> What's the outstanding question, exactly?  It seems to me we should
> just rename them.

Ah, I misunderstood before.

This ended up being a bit more work on the LLVM side, but the attached
patch replaces all of the ptx.read intrinsics with nvvm.read.ptx.sreg
variants, cleans up the duplicate nvvm variants that leaves around, and
renames ptx.bar.sync to nvvm.bar.sync.

The clang patch just updates the names and doesn't change anything with
this version.

WDYT?

-------------- next part --------------
A non-text attachment was scrubbed...
Name: llvm-remove-ptx-intrins-2.patch
Type: text/x-patch
Size: 40355 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160706/957fe919/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: clang-remove-ptx-intrins-2.patch
Type: text/x-patch
Size: 17751 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160706/957fe919/attachment-0001.bin>