This revision was automatically updated to reflect the committed changes.
Closed by commit rG71b06585857a: [NVPTX] Add f16 and v2f16 ldg builtins
(authored by jchlanda).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D144961/new/
https://reviews.llvm.
jchlanda added inline comments.
Comment at: llvm/test/CodeGen/NVPTX/ldu-ldg.ll:5-6
declare i8 @llvm.nvvm.ldu.global.i.i8.p1(ptr addrspace(1) %ptr, i32 %align)
declare i32 @llvm.nvvm.ldu.global.i.i32.p1(ptr addrspace(1) %ptr, i32 %align)
+
tra wrote:
> Do we w
jchlanda updated this revision to Diff 501392.
jchlanda added a comment.
Add more verbose test names.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D144961/new/
https://reviews.llvm.org/D144961
Files:
clang/include/clang/Basic/BuiltinsNVPTX.def
clang/lib/CodeGen/CGBuiltin.cpp
clan
tra accepted this revision.
tra added inline comments.
This revision is now accepted and ready to land.
Comment at: llvm/test/CodeGen/NVPTX/ldu-ldg.ll:5-6
declare i8 @llvm.nvvm.ldu.global.i.i8.p1(ptr addrspace(1) %ptr, i32 %align)
declare i32 @llvm.nvvm.ldu.global.i.i32.p1(pt
jchlanda created this revision.
jchlanda added a reviewer: tra.
Herald added subscribers: mattd, gchakrabarti, asavonic.
Herald added a project: All.
jchlanda requested review of this revision.
Herald added subscribers: llvm-commits, cfe-commits, jholewinski.
Herald added projects: clang, LLVM.
Ad