Mordante added a comment. In D150450#4337952 <https://reviews.llvm.org/D150450#4337952>, @erichkeane wrote:
> In D150450#4337858 <https://reviews.llvm.org/D150450#4337858>, @cor3ntin > wrote: > >> Similar PR here https://reviews.llvm.org/D149875 > > Ah! I'd not noticed that one (or maybe someone mentioned it?). If @Mordante > wants to take this over/change his to meet our RFC, I'm all for it. I've mentioned it in https://discourse.llvm.org/t/rfc-lets-just-call-it-c-26-and-forget-about-the-c-2c-business-at-least-internally/70383/3 to avoid duplicated effort. It seems, as expected. the patches have a large overlap, but you added a bit more of the status page and I updated the clang-cl driver. I just updated my patch before I saw this update in the mail. ================ Comment at: clang/lib/Frontend/InitPreprocessor.cpp:457 + Builder.defineMacro("__cplusplus", "202402L"); + g/ FIXME: Use correct value for C++23, and update C++26 to be 'one more'. + else if (LangOpts.CPlusPlus23) ---------------- Note this has been committed in the draft for N4950. https://github.com/cplusplus/draft/commit/4e4de1df8ee941255b653b61d0a62050b34cf8c9#diff-1b14eeb3be43fba46b5d0ff51fe0695c9a856544d1f45b1d486f98f5e1b889bf CHANGES SINCE LAST ACTION https://reviews.llvm.org/D150450/new/ https://reviews.llvm.org/D150450 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
