[PATCH] D150450: Add C++26 compile flags.

Fri May 12 09:02:53 PDT 2023

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