[llvm] [Offload] Make CUDA Driver Version a string (PR #146049)
via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 27 03:45:24 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-offload
Author: Ross Brunton (RossBrunton)
<details>
<summary>Changes</summary>
AMD treats this value as a string, so for consistency require this in
NVIDIA as well. This shouldn't change the output of the
`llvm-offload-device-info` tool, but does fix an issue in liboffload
when it tries to query the version.
---
Full diff: https://github.com/llvm/llvm-project/pull/146049.diff
1 Files Affected:
- (modified) offload/plugins-nextgen/cuda/src/rtl.cpp (+3-1)
``````````diff
diff --git a/offload/plugins-nextgen/cuda/src/rtl.cpp b/offload/plugins-nextgen/cuda/src/rtl.cpp
index 0e662b038c363..f1164074f9ea9 100644
--- a/offload/plugins-nextgen/cuda/src/rtl.cpp
+++ b/offload/plugins-nextgen/cuda/src/rtl.cpp
@@ -930,7 +930,9 @@ struct CUDADeviceTy : public GenericDeviceTy {
CUresult Res = cuDriverGetVersion(&TmpInt);
if (Res == CUDA_SUCCESS)
- Info.add("CUDA Driver Version", TmpInt);
+ // For consistency with other drivers, store the version as a string
+ // rather than an integer
+ Info.add("CUDA Driver Version", std::to_string(TmpInt));
Info.add("CUDA OpenMP Device Number", DeviceId);
``````````
</details>
https://github.com/llvm/llvm-project/pull/146049
More information about the llvm-commits
mailing list