jdoerfert added a comment.

In D74571#1877000 <https://reviews.llvm.org/D74571#1877000>, @kkwli0 wrote:

> > I am not sure I understand. Do we need to modify stuff outside of 
> > `/openmp`? I was hoping it is our CMake that can be adjusted to make this 
> > work as described earlier. TBH, he warning is even not my biggest problem. 
> > As long as we get a libomptarget.bc we should be fine.
>
> @jdoerfert  Yes.  The warning message affects the outcome of another test in 
> cmake.  With the approach of ignoring the warning message, we need to modify 
> stuff outside of `/openmp`.  For me, I don't mind to have the warning 
> messages all over the places.  The problem we have here is unable to build 
> libomp.so.


I was naively hoping we can say `-Wno-unknown-cuda-version` somewhere "early" 
in `openmp/CMake...` to ensure we can build the runtime.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74571/new/

https://reviews.llvm.org/D74571



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to