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

Tobias Hieta via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 28 01:18:24 PDT 2022


thieta added a comment.

In D130689#3684330 <https://reviews.llvm.org/D130689#3684330>, @mehdi_amini wrote:

> What does it mean exactly? We can't use **anything** C++17 without writing it in the coding standards?
> I'm not sure it'll be manageable: how do you see this playing out?

Probably poorly worded - what I was trying to convey is that if we want to use a C++17 feature that's really impactful on the syntax/readability we should probably consider recommending ways to use it in the coding standards, similar to our guidelines on using for() loops (https://llvm.org/docs/CodingStandards.html#use-range-based-for-loops-wherever-possible) or the auto keyword (https://llvm.org/docs/CodingStandards.html#use-auto-type-deduction-to-make-code-more-readable).


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