Mordante added a comment.

Thanks for the patch!



================
Comment at: clang/lib/Headers/stdatomic.h:21
+#if __STDC_HOSTED__ && __has_include_next(<stdatomic.h>)                       
 \
+    && !(defined(_MSC_VER) && __cplusplus-0 < 202002l)
 # include_next <stdatomic.h>
----------------
Is `__cplusplus-0` intentional? If so please add some comments how this differs 
from `__cplusplus`.
Since this is a C header we should test whether the macro exists before 
querying its value.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139266

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

Reply via email to