[PATCH] D108247: [CUDA] Improve CUDA version detection and diagnostics.

Jonas Hahnfeld via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 18 10:55:58 PDT 2021


Hahnfeld accepted this revision.
Hahnfeld added a comment.
This revision is now accepted and ready to land.

Otherwise LGTM



================
Comment at: clang/lib/Driver/ToolChains/Cuda.cpp:103-104
+    std::string VersionString = CudaVersionToString(Version);
+    if (!VersionString.empty())
+      VersionString += " ";
+    D.Diag(diag::warn_drv_new_cuda_version)
----------------
I think you have to prepend the space in order to match `CUDA version%0 is newer than`, or move the `%0`. But I didn't try locally, might be missing something.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108247



More information about the cfe-commits mailing list