[PATCH] D95974: [CUDA, NVPTX] Allow targeting sm_86 GPUs.

2021-02-09 Thread Artem Belevich via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG2aa01ccec301: [CUDA, NVPTX] Allow targeting sm_86 GPUs. (authored by tra). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https:

[PATCH] D95974: [CUDA, NVPTX] Allow targeting sm_86 GPUs.

2021-02-03 Thread Johannes Doerfert via Phabricator via cfe-commits
jdoerfert added a comment. In D95974#2540352 , @tra wrote: >> So we add ptx72 but it's not used with sm_86, interesting. > > `ptx71` is the minimum/default requited PTX version for sm_86. If we compile > with CUDA-11.2, clang will set the '+ptx72' as we m

[PATCH] D95974: [CUDA, NVPTX] Allow targeting sm_86 GPUs.

2021-02-03 Thread Artem Belevich via Phabricator via cfe-commits
tra marked an inline comment as done. tra added a comment. > So we add ptx72 but it's not used with sm_86, interesting. `ptx71` is the minimum/default requited PTX version for sm_86. If we compile with CUDA-11.2, clang will set the '+ptx72' as we may potentially need it in order to link in libd

[PATCH] D95974: [CUDA, NVPTX] Allow targeting sm_86 GPUs.

2021-02-03 Thread Artem Belevich via Phabricator via cfe-commits
tra updated this revision to Diff 321220. tra edited the summary of this revision. tra added a comment. Removed debug printout Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95974/new/ https://reviews.llvm.org/D95974 Files: clang/include/clang/Ba

[PATCH] D95974: [CUDA, NVPTX] Allow targeting sm_86 GPUs.

2021-02-03 Thread Johannes Doerfert via Phabricator via cfe-commits
jdoerfert accepted this revision. jdoerfert added a comment. This revision is now accepted and ready to land. One nit below. Seems reasonable in general. So we add ptx72 but it's not used with sm_86, interesting. Comment at: clang/lib/Driver/ToolChains/Cuda.cpp:103 Line

[PATCH] D95974: [CUDA, NVPTX] Allow targeting sm_86 GPUs.

2021-02-03 Thread Artem Belevich via Phabricator via cfe-commits
tra created this revision. tra added reviewers: yaxunl, jdoerfert. Herald added subscribers: dexonsmith, bixia, hiraditya, jholewinski. tra requested review of this revision. Herald added projects: clang, LLVM. The patch only plumbs through the option necessary for targeting sm_86 GPUs w/o adding