kkwli0 added a comment. In D75001#1922789 <https://reviews.llvm.org/D75001#1922789>, @jdoerfert wrote:
> In D75001#1891284 <https://reviews.llvm.org/D75001#1891284>, @Hahnfeld wrote: > > > In D75001#1887876 <https://reviews.llvm.org/D75001#1887876>, @jdoerfert > > wrote: > > > > > I like this way better. I was hoping we could do it in our cmake only :) > > > > > > Give others a day or so to comment before your commit but I'm fine with > > > this. > > > > > > Well, that doesn't really work if `openmp-commits` is only subscribed on > > commit. That said, the solution is a bit ugly but I don't have an > > alternative right now. > > > What is the problem with `openmp-commits` here? I got the emails, didn't you? > > @kkwli0 Where are we with this? @jdoerfert In terms of the scope of this issue, it is done. It is to ignore the warning in order to avoid build blockage in the future when newer CUDA toolkit is available and installed on the build system. Another related issue that blocked the other part of the tests (related to python binding) was resolved in https://reviews.llvm.org/D76030. As far as I know, the remaining one (not quite an issue, it is by design) is warning issued when -fopenm-targets=nvptx* is specified, users can suppress it. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75001/new/ https://reviews.llvm.org/D75001 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
