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

[Mehdi AMINI via Phabricator via cfe-commits]

mehdi_amini added a comment.

In D130689#3684333 <https://reviews.llvm.org/D130689#3684333>, @thieta wrote:

> 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).

Makes sense, thanks! Let's just update the wording to convey this and this all looks good to me :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130689