tra added a comment.
Herald added a subscriber: dexonsmith.

This patch is also missing corresponding LLVM changes that define features for 
sm_86 and ptx71. 
See 
https://github.com/llvm/llvm-project/blob/master/llvm/lib/Target/NVPTX/NVPTX.td



================
Comment at: clang/lib/Driver/ToolChains/Cuda.cpp:158-160
+      // CUDA 7.0 and 11.1 don't have a version.txt, so try to guess
+      // if version.txt isn't present.
+      Version = CudaVersion::CUDA_111;
----------------
This should not be needed any more. Clang now extracts CUDA version from cuda.h.


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