erichkeane marked 8 inline comments as done. erichkeane added inline comments.
================ Comment at: clang/docs/CommandGuide/clang.rst:206-210 + Working draft for ISO C++ 2026 + + | ``gnu++2c`` + + Working draft for ISO C++ 2026 with GNU extensions ---------------- aaron.ballman wrote: > We shouldn't call it `ISO C++ 2026` until it's published by ISO. (I don't > have strong opinions about 2c vs 26 in there, but since we need to update it > anyway when 26 is released, I figured it'd be easier to find again with 2c.) Ah, I just copied from what we had for C++23 before the flip. But this makes sense, I chose `2c` for the reason you mentioned. ================ Comment at: clang/lib/Frontend/InitPreprocessor.cpp:456 + // FIXME: Use correct value for C++26. + Builder.defineMacro("__cplusplus", "202402L"); + else if (LangOpts.CPlusPlus23) ---------------- aaron.ballman wrote: > This value is fine by me, but we should probably coordinate with the GCC > folks so we don't wind up with two different values between us. Email sent to GCC maintainer (aaron copied!). ================ Comment at: clang/www/cxx_status.html:1568 +<h2 id="cxx26">C++2c implementation status</h2> + ---------------- aaron.ballman wrote: > cor3ntin wrote: > > h-vetinari wrote: > > > The longer this page gets, the more I feel it should be sorted in reverse > > > chronological order of the standard versions. It's bothersome IMO to keep > > > scrolling past almost-fully implemented standards (currently the tables > > > for C++17 & C++20) before getting to see the status of the latest & > > > greatest. > > > > > > https://en.cppreference.com/w/cpp/compiler_support is sorted in reverse > > > order as well. > > This sounds like an excellent improvement. > > Erich, would you rather do that in this pr, or i can submit a different > > patch afterwards? > I love the idea, and I think it should be done in a separate patch. Corentin: you're welcome to do it in a different patch, but I like the idea as well! 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