cor3ntin added inline comments.
================ Comment at: clang/docs/ReleaseNotes.rst:80 +- Removed the ``ATOMIC_VAR_INIT`` macro in C2x and later standards modes, which + implements `WG14 N2886 <https://www.open-std.org/jtc1/sc22/wg14/www/docs/n2886.htm>`_ ---------------- Just confirming we want to pluralize standards here? It kind of look weird to my non native eyes. ================ Comment at: clang/lib/Headers/stdatomic.h:50 + in C2x mode; switch to the correct values once they've been published. */ +#if (defined(__STDC_VERSION__) && __STDC_VERSION__ < 202000L) || \ + defined(__cplusplus) ---------------- C++ uses the date of the meeting where the change was accepted, I assume C is different? ================ Comment at: clang/lib/Headers/stdatomic.h:61 /* ATOMIC_VAR_INIT was deprecated in C17 and C++20. */ #pragma clang deprecated(ATOMIC_VAR_INIT) #endif ---------------- Should we add a message informing people it's remove in C23? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144196/new/ https://reviews.llvm.org/D144196 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
