[PATCH] D55269: [CUDA] Fix nvidia-cuda-toolkit detection on Ubuntu
Joel E. Denny via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Dec 6 10:09:05 PST 2018
jdenny added a comment.
In D55269#1320382 <https://reviews.llvm.org/D55269#1320382>, @tra wrote:
> >> Let's start with fixing OpenMP's cmake files. Once it no longer insists on specifying --cuda-path=/usr, and isUbuntu is in place, what is the remaining failure that you see?
> >
> > I disagree here: It's not OpenMP but CMake that's detecting the wrong path here. This is the place to fix it once and for all (and possibly get the shim package right in that process).
>
> It's somewhat orthogonal, IMO. I agree that it's not OpenMP's problem. Cmake will fix it at some point in future, but, presumably, we want OpenMP buildable *now*. Adding a temporary workaround for something that Cmake can't handle now would make building OpenMP with clang somewhat easier which was the end goal of this patch. In the end it's up to OpenMP maintainers what they would want to do.
I don't know when I'll explore the cmake problem further. If someone else decides to, please cc/subscribe me.
Thanks for the reviews.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55269/new/
https://reviews.llvm.org/D55269
More information about the cfe-commits
mailing list