tra added inline comments.

================
Comment at: clang/lib/Headers/__clang_cuda_builtin_vars.h:31
+#ifdef __OPENMP_NVPTX__
+#define DEVICE
+#else
----------------
You should use `__` prefix to avoid unintentional clashes with user-defined 
macros.
`__DEVICE__` ?


================
Comment at: clang/lib/Headers/__clang_cuda_builtin_vars.h:124
 #undef __CUDA_DISALLOW_BUILTINVAR_ACCESS
 #undef __DELETE
 
----------------
You need to `#undef` the DEVICE macro here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94884

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

Reply via email to