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

LG



================
Comment at: clang/lib/Headers/openmp_wrappers/complex:20
 
+#ifdef __NVPTX__
 #define __OPENMP_NVPTX__
----------------
JonChesterfield wrote:
> I don't think this should be necessary - the declare variant stuff works for 
> nvptx afaik
You need it because the file has include guards.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108552

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to