tra requested changes to this revision.
tra added a comment.
This revision now requires changes to proceed.

It's a bit premature to call CUDA-10.2 supported. We can compile using it, but 
clang/llvm has no support for the new things introduced by CUDA-10.2. 
E.g. CUDA-10.2 introduces new PTX version with new instructions (and matching 
clang builtins) 
https://docs.nvidia.com/cuda/parallel-thread-execution/index.html#ptx-isa-version-6-5

Any code which checks cuda version (and gets 10.2) and tries to use those new 
features will fail.
IMO the warning does serve a valid purpose here indicating that we only intend 
to provide features available in 10.1.

Given that clang provides an option to disable the warning, I don't think this 
change should be applied.


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

https://reviews.llvm.org/D74571



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

Reply via email to