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

[Nikita Popov via Phabricator via cfe-commits]

nikic added a comment.

In D131465#3776537 <https://reviews.llvm.org/D131465#3776537>, @MaskRay wrote:

> In D131465#3776515 <https://reviews.llvm.org/D131465#3776515>, @nikic wrote:
>
>> I've reverted this change because it causes major llvm-test-suite breakage. You likely need to pin many tests to use `-std=c++14`.
>
> https://github.com/llvm/llvm-test-suite/commit/b3a89445dc4aed7ee9825e024c32d417e36a3f13 should fix llvm-test-suite.

Thanks!

It looks like the switch to `-std=c++17` has some major compile-time impact: http://llvm-compile-time-tracker.com/compare.php?from=047c7aa96dadf8a2c71a29e2df610d628d9e7e3e&to=3e99b8d947ac024831e59be2b604ac67a24fed94&stat=instructions The `O0` build of 7zip becomes 80% slower (without codegen changes).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131465