[PATCH] D90415: [OpenMP] Use __OPENMP_NVPTX__ instead of _OPENMP in complex wrapper headers.

Johannes Doerfert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 29 13:06:01 PDT 2020


jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.

You need to define & undefine the macro around the includes of `__clang_cuda_complex_builtins.h` in `clang/lib/Headers/openmp_wrappers/complex` and `clang/lib/Headers/openmp_wrappers/complex.h`. (see also rG7f1e6fcff942 <https://reviews.llvm.org/rG7f1e6fcff9427adfa8efa3bfeeeac801da788b87>) That should fix the tests. Assuming the tests pass, LGTM.

For release 11, can you file a bug and prepare a patch that applies to the relevant branch?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D90415/new/

https://reviews.llvm.org/D90415



More information about the cfe-commits mailing list