This revision was automatically updated to reflect the committed changes.
tra marked an inline comment as done.
Closed by commit rL278549: [CUDA] Place GPU binary into .nv_fatbin section and
align it by 8. (authored by tra).
Changed prior to commit:
https://reviews.llvm.org/D23429?vs=67851&id=6
tra marked an inline comment as done.
Comment at: lib/CodeGen/CGCUDANV.cpp:62-69
@@ -60,3 +61,10 @@
llvm::ConstantInt::get(SizeTy, 0)};
auto ConstStr = CGM.GetAddrOfConstantCString(Str, Name.c_str());
+llvm::GlobalVariable *GV =
+ca
tra updated this revision to Diff 67851.
tra added a comment.
Reverted argument type to std::string
https://reviews.llvm.org/D23429
Files:
lib/CodeGen/CGCUDANV.cpp
test/CodeGenCUDA/device-stub.cu
Index: test/CodeGenCUDA/device-stub.cu
===
jlebar added inline comments.
Comment at: lib/CodeGen/CGCUDANV.cpp:62
@@ -60,2 +61,3 @@
llvm::ConstantInt::get(SizeTy, 0)};
-auto ConstStr = CGM.GetAddrOfConstantCString(Str, Name.c_str());
+auto ConstStr = CGM.GetAddrOfConstantCString(Str,
tra created this revision.
tra added a reviewer: jlebar.
tra added a subscriber: cfe-commits.
This matches the way nvcc encapsulates GPU binaries into host object file.
Now cuobjdump can deal with clang-compiled object files.
https://reviews.llvm.org/D23429
Files:
lib/CodeGen/CGCUDANV.cpp
