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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to