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

Reply via email to