This is an archive of the discontinued LLVM Phabricator instance.

[Libomptarget] Remove find_package(CUDA) as it has been deprecated
ClosedPublic

Authored by jhuber6 on Jan 25 2023, 11:21 AM.

Details

Summary

Since D137724 and the LLVM 17 release we have updated to CMake version
3.20. This means that find_package(CUDA) is officially deprecated and
can be replaced with find_package(CUDAToolkit) instead. This patch
does this and also cleans up a bit of the CMake.

Diff Detail

Event Timeline

jhuber6 created this revision.Jan 25 2023, 11:21 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 25 2023, 11:21 AM
Herald added subscribers: mattd, yaxunl. · View Herald Transcript
jhuber6 requested review of this revision.Jan 25 2023, 11:21 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 25 2023, 11:21 AM

This patch contains the cleanup and deprecation of old CMake find package stuff. Better to split them.

This patch contains the cleanup and deprecation of old CMake find package stuff. Better to split them.

This required changing it to CUDA::cuda_driver in the link libs. So I figured I'd just clean it up while I was there. It'd be a little annoying to split them but I can do that if necessary.

This patch contains the cleanup and deprecation of old CMake find package stuff. Better to split them.

This required changing it to CUDA::cuda_driver in the link libs. So I figured I'd just clean it up while I was there. It'd be a little annoying to split them but I can do that if necessary.

It is generally preferred to only do one thing in one patch if possible.

jhuber6 updated this revision to Diff 492202.Jan 25 2023, 11:37 AM

Moving cleanup to a follow-up patch

This revision is now accepted and ready to land.Jan 25 2023, 11:39 AM