[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