[PATCH] D50207: [NVPTX] Handle __nvvm_reflect("__CUDA_ARCH").

Artem Belevich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 2 17:13:52 PDT 2018


tra added a comment.

In https://reviews.llvm.org/D50207#1186491, @jlebar wrote:

> Just to check, the notion is that it's OK if I report a sm version less than what I end up running on?


Yes. We may lose some performance, but not correctness as we are expected to be forward-compatible.
Until now reflect was being replaced with 0, so we were picking the variant suitable for the oldest GPU.


https://reviews.llvm.org/D50207





More information about the llvm-commits mailing list