JonChesterfield added inline comments.
================ Comment at: clang/lib/Headers/openmp_wrappers/complex:21 #define __OPENMP_NVPTX__ #include <__clang_cuda_complex_builtins.h> #undef __OPENMP_NVPTX__ ---------------- ^ this header does not look for a macro called __CUDA__ or include any other headers so I believe dropping the macro can make no change to that header. It might affect other things that happen to be included after this header, but iiuc cuda and openmp-nvptx both define `__CUDA__` anyway, so that could only break amdgpu applications that were erroneously looking for a cuda macro. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D105221/new/ https://reviews.llvm.org/D105221 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits