[PATCH] D131465: C++/ObjC++: switch to gnu++17 as the default standard

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 6 08:15:51 PDT 2022


aaron.ballman added a comment.

Precomit CI found valid failures that seem plausibly related to your changes, but Windows and Debian found different failures which is a bit of a surprise to me. Can you check those?



================
Comment at: clang/test/lit.site.cfg.py.in:27
 config.clang_enable_opaque_pointers = @CLANG_ENABLE_OPAQUE_POINTERS_INTERNAL@
+config.clang_default_std_cxx = "@CLANG_DEFAULT_STD_CXX@"
 config.clang_default_cxx_stdlib = "@CLANG_DEFAULT_CXX_STDLIB@"
----------------
Should we add some documentation for this in a follow-up? (I know `CLANG_DEFAULT_STD_CXX` already exists, but it seems like it'd be helpful to tell users about it too.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131465



More information about the cfe-commits mailing list