This revision was automatically updated to reflect the committed changes.
Closed by commit rL331938: [CUDA] Added -f[no-]cuda-short-ptr option (authored
by tra, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D46148?vs=144419&id=146027#
tra updated this revision to Diff 144419.
tra added a subscriber: arsenm.
tra added a comment.
Do not use subtarget feature to control codegen of short pointers.
Instead we need to add a field to TargetOptions which is the only way to pass
this info to NVPTXTargetInfo constructor where we calcula
bkramer accepted this revision.
bkramer added a comment.
This revision is now accepted and ready to land.
lgtm
https://reviews.llvm.org/D46148
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe
tra updated this revision to Diff 144216.
tra added a comment.
Removed debug printout.
https://reviews.llvm.org/D46148
Files:
clang/include/clang/Driver/Options.td
clang/lib/Basic/Targets/NVPTX.cpp
clang/lib/Driver/ToolChains/Cuda.cpp
Index: clang/lib/Driver/ToolChains/Cuda.cpp
tra created this revision.
tra added a reviewer: jlebar.
Herald added subscribers: bixia, sanjoy, jholewinski.
The option enables use of 32-bit pointers for accessing
const/local/shared memory. The feature is disabled by default.
https://reviews.llvm.org/D46148
Files:
clang/include/clang/Driv
