[PATCH] D88668: [CUDA] Add support for 11.1

Artem Belevich via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 1 09:42:37 PDT 2020


tra added a comment.

Generally speaking the default should be conservative. It does us no good if we generate PTX 99.99, but discover that ptxas does not support it. Granted, these days 7.0 is also the wrong default as it's pretty ancient. IMO bumping it to 9.0 and GPU arch to sm_30 would be sensible.

Now, as for the missing version file, we should probably allow explicitly specifying the version. We already do it for HIP.



================
Comment at: clang/lib/Driver/ToolChains/Cuda.cpp:662
+  case CudaVersion::CUDA_111:
+    PtxFeature = "+ptx71";
+    break;
----------------
I do not think LLVM has this feature implemented. I'm not sure how it reacts to clang specifying non-existing feature.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D88668/new/

https://reviews.llvm.org/D88668



More information about the cfe-commits mailing list