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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
