Compiler error at CUDA Plugin pi_cuda.cpp

[jandrovins]

Describe the bug
While compiling from source, an error occurred while compiling llvm/sycl/plugins/cuda/pi_cuda.cpp.

llvm/sycl/plugins/cuda/pi_cuda.cpp:1957:35: error: use of undeclared identifier 'cuDeviceGetUuid_v2'; did you mean 'cuDeviceGetUuid'? sycl::detail::pi::assertion(cuDeviceGetUuid_v2(&uuid, device->get()) == ^~~~~~~~~~~~~~~~~~ cuDeviceGetUuid

To Reproduce

1. Run configure.py: CUDA_LIB_PATH=/usr/local/cuda-10.2/lib64/stubs/ CC=clang CXX=clang++ python $DPCPP_HOME/llvm/buildbot/configure.py --cuda --cmake-opt="-DCUDA_TOOLKIT_ROOT_DIR=/usr/local/cuda-10.2" --cmake-gen "Ninja" -o ./build-ibm --host-target PowerPC --use-lld
2. Try to compile: CUDA_LIB_PATH=/usr/local/cuda-10.2/lib64/stubs/ CC=clang CXX=clang++ python $DPCPP_HOME/llvm/buildbot/compile.py -o ./build-ibm -j 40
3. While compiling the error above occurred. By reading the relevant code snippet I guess that the reason for this error is that since I do not have a version of CUDA above 11.4 then cuDeviceGetUuid_v2 is not available at compile time.

Environment (please complete the following information):

• OS: RHEL 7.5
• Target device and vendor: IBM Power CPU and Nvidia Volta V100.
• DPC++ version: Commit hash from git log: 10e07a3
• Dependencies version: cuda 10.2

Additional context
N/A

[steffenlarsen]

Thank you for reporting this, @jandrovins!

Looks like the dependence on cuDeviceGetUuid_v2 was introduced in #8203. @npmiller - Do you think it would be sufficient if we change the conditional picking the version of cuDeviceGetUuid to use to instead be based on the CUDA_VERSION macro?

[npmiller]

Oh that's a good catch, yeah I think it should be sufficient to use a version macro instead.

[zjin-lcf]

Thank you for reporting the issue. User may be unwilling to upgrade their CUDA SDK.
Is it ok to remove the "v2" call and call the "v1" for any Cuda version? Thanks.

[steffenlarsen]

#8765 should have addressed this issue. @jandrovins - Can you please confirm?

[jandrovins]

This issue is solved, compilation works well now.

[steffenlarsen]

Thank you, @jandrovins! Closing this as fixed with #8765.