[PATCH] D130689: [LLVM] Update C++ standard to 17

Tobias Hieta via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 29 01:02:15 PDT 2022


thieta added a comment.

In D130689#3686397 <https://reviews.llvm.org/D130689#3686397>, @thakis wrote:

> It'd be nice if this was landed opt-in behind some cmake variable at first, so that folks could try it out on their bots and see how well things work.

You can already test this with `-DCMAKE_CXX_STANDARD=17` afaik. I wonder how many bot owners would actually test this if we made another flag available.

Since we can already test this with a current flag maybe we should just post to discourse that bot-owners can test it and a date when this will be merged. But I don't expect this to be a big problem, when we merged the soft error earlier this year - it only broke one bot which was out of date.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130689



More information about the cfe-commits mailing list