gtbercea abandoned this revision.
gtbercea added a comment.
Functionality already landed. See previous comment.
Repository:
rL LLVM
https://reviews.llvm.org/D41486
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
Hahnfeld removed a reviewer: Hahnfeld.
Hahnfeld added a comment.
Not needed anymore, we decided on the order of arguments in
https://reviews.llvm.org/D41012.
Repository:
rL LLVM
https://reviews.llvm.org/D41486
___
cfe-commits mailing list
gtbercea updated this revision to Diff 127865.
gtbercea added a comment.
Address comments.
Repository:
rL LLVM
https://reviews.llvm.org/D41486
Files:
lib/CodeGen/CGOpenMPRuntimeNVPTX.cpp
test/OpenMP/nvptx_data_sharing.cpp
test/OpenMP/nvptx_target_teams_codegen.cpp
Index:
gtbercea added a comment.
In https://reviews.llvm.org/D41486#961981, @Hahnfeld wrote:
> https://reviews.llvm.org/D41012? This patch doesn't update the documentation
> with function signatures.
Ok so I see that your patch uses a different order of the arguments. I've just
added the data
Hahnfeld requested changes to this revision.
Hahnfeld added a comment.
This revision now requires changes to proceed.
https://reviews.llvm.org/D41012? This patch doesn't update the documentation
with function signatures.
Repository:
rL LLVM
https://reviews.llvm.org/D41486
gtbercea created this revision.
gtbercea added reviewers: ABataev, carlo.bertolli, hfinkel, Hahnfeld, caomhin.
Herald added a subscriber: jholewinski.
This patch adds a missing argument to the runtime interface. Tests are adjusted
accordingly.
Repository:
rL LLVM
