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

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 9 10:45:05 PDT 2022


MaskRay added a comment.

In D131465#3780717 <https://reviews.llvm.org/D131465#3780717>, @AaronLiu wrote:

> This cause a large amount failures in our testing with errors such as:
>
>   error: ISO C++17 does not allow 'register' storage class specifier [-Wregister]
>
>   error: ISO C++17 does not allow dynamic exception specifications [-Wdynamic-exception-spec]

You may want to `-std=gnu++14` or `-std=c++14` to pin your projects to C++14, instead of relying on the Clang default.
Is your case is like PlayStation 4 and 5 where a platform wants to have a different default (I'd hope that we can try avoiding such differences), a target maintainer can weigh in and perhaps change it.
(The configuration file discussions are interesting, too https://discourse.llvm.org/t/configuration-files/42529 https://discourse.llvm.org/t/rfc-adding-a-default-file-location-to-config-file-support/63606)


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