aaron.ballman added a comment. In D146497#4216197 <https://reviews.llvm.org/D146497#4216197>, @aaron.ballman wrote:
> In D146497#4215650 <https://reviews.llvm.org/D146497#4215650>, @Hahnfeld > wrote: > >> FYI this commit breaks `clang/test/Index/index-file.cu` for me: >> >> warning: CUDA version 11.8 is only partially supported >> [-Wunknown-cuda-version] >> warning: CUDA version 11.8 is only partially supported >> [-Wunknown-cuda-version] >> /home/jhahnfel/LLVM/src/clang/test/Index/index-file.cu:8:20: error: >> CHECK-HOST-NOT: excluded string found in input >> // CHECK-HOST-NOT: macro definition=__CUDA_ARCH__ >> ^ >> <stdin>:6218:48: note: found here >> // CHECK: __clang_cuda_runtime_wrapper.h:70:9: macro >> definition=__CUDA_ARCH__ Extent=[70:9 - 70:27] >> >> ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ >> >> It //could// be related to the warning about my locally installed CUDA >> version, but `__CUDA_ARCH__` should really not be defined when compiling for >> the host. If you have any idea what's going on here, please let me know - >> I'm a bit puzzled right now... > > This is confirmed on at least this build bot: > https://lab.llvm.org/buildbot/#/builders/91/builds/15271 > > I'll investigate locally and revert if it's not something I can fix forward. The issue doesn't reproduce for me locally on my Windows machine, so it may be something specific to the VE setup. It looks like the macro is being printed on the host side when it's expected to not be present, which doesn't seem like an expected outcome from this change, so I reverted in 18d56880a89ad7d58f8543d148facebd079cef19 <https://reviews.llvm.org/rG18d56880a89ad7d58f8543d148facebd079cef19> to hopefully bring the bot back to green (if the bot stays red for the same reason, I'll reapply the changes). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146497/new/ https://reviews.llvm.org/D146497 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
