[PATCH] D141861: [nvptx-arch] Dynamically load the CUDA runtime if not found during the build
Steven Johnson via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jan 30 13:56:48 PST 2023
srj added a comment.
In D141861#4091869 <https://reviews.llvm.org/D141861#4091869>, @jhuber6 wrote:
> In D141861#4091851 <https://reviews.llvm.org/D141861#4091851>, @srj wrote:
>
>>> https://github.com/llvm/llvm-project/commit/759dec253695f38a101c74905c819ea47392e515. Does it work if you revert this? I wouldn't think it wouldn't affect anything. That's the only change that happened after the 16 release as far as I'm aware.
>>
>> Reverting this (well, actually just monkey-patching in the old code) does indeed make things build correctly again -- a bit surprising to me too, but that's CMake for you. Reverting the change seems appropriate pending a tested fix.
>
> That's bizarre, that means it's finding `CUDAToolkit` but can't link against it?
It's finding a 64-bit CUDAToolkit, which it can't link against because the rest of the build is 32-bit.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D141861/new/
https://reviews.llvm.org/D141861
More information about the cfe-commits
mailing list