gtbercea closed this revision.
gtbercea added a comment.
Committed here https://reviews.llvm.org/D41123
Repository:
rL LLVM
https://reviews.llvm.org/D40451
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mail
hfinkel accepted this revision.
hfinkel added a comment.
This revision is now accepted and ready to land.
LGTM
Comment at: lib/CodeGen/CGOpenMPRuntimeNVPTX.cpp:937
if (!CapturedVars.empty()) {
+ // There's somehting to share, add the attribute
+ CGF.CurFn->addFnA
gtbercea updated this revision to Diff 124246.
Repository:
rL LLVM
https://reviews.llvm.org/D40451
Files:
lib/CodeGen/CGOpenMPRuntimeNVPTX.cpp
test/OpenMP/nvptx_data_sharing.cpp
Index: test/OpenMP/nvptx_data_sharing.cpp
===
gtbercea created this revision.
Herald added a subscriber: jholewinski.
Since OpenMP and CUDA share the same toolchain we need to disable:
- the lowering of variables to shared memory in the LLVM NVPTX backend
- the emission of the shared depot
- the emission of shared stack pointers
when compil