[PATCH] D74571: [OpenMP][CUDA] Add CUDA 10.2 support

Johannes Doerfert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 14 20:35:08 PST 2020


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





More information about the cfe-commits mailing list