jdoerfert added a comment. In D77240#1955755 <https://reviews.llvm.org/D77240#1955755>, @tra wrote:
> In D77240#1955724 <https://reviews.llvm.org/D77240#1955724>, @jdoerfert wrote: > > > At least that one is defined in what is "now" `__clang_cuda_math.h`: > > > Cool. We may be OK, but we still need to verify it. Math headers are rather > fragile and we need to make sure it all still works two different standard > libraries and all CUDA versions. > Unfortunately my CUDA build bot has decided to die the day we've started > working from home, so it has to be done manually. > Let me try patching in your changes and try them with the CUDA versions I > have. Stay tuned. Thank you very much! I (now) know what pain math headers are... These are almost the last infrastructure changes I need, there is a problem with the include path order on some system but for that I first need to write a patch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77240/new/ https://reviews.llvm.org/D77240 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
