[PATCH] D139266: Headers: tweak inclusion condition for stdatomic.h

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 6 10:29:13 PST 2022


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

Thanks for working on this! The changes LGTM. I've been trying to figure out if there's a way we can add test coverage for this change, but I'm really not coming up with much. However, the changes definitely should have a release note.



================
Comment at: clang/lib/Headers/stdatomic.h:23
+    __has_include_next(<stdatomic.h>) &&                                       \
+    !(defined(_MSC_VER) && defined(__cplusplus) && __cplusplus < 202002l)
 # include_next <stdatomic.h>
----------------
`202002l` -> `202002L` to make it easier on people's eyeballs.


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

https://reviews.llvm.org/D139266



More information about the cfe-commits mailing list