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

Tue Aug 9 10:01:21 PDT 2022

aaron.ballman added a comment.

In D130689#3710281 <https://reviews.llvm.org/D130689#3710281>, @royjacobson wrote:

> In D130689#3709834 <https://reviews.llvm.org/D130689#3709834>, @thieta wrote:
>
>> In D130689#3709742 <https://reviews.llvm.org/D130689#3709742>, @aaron.ballman wrote:
>>
>>> One thing I think would be a definite improvement is to have done an RFC on Discourse for these changes so that downstreams have a chance to weigh in on the impact. The patch was put up on Jul 28 and landed about a week later without any notification to the rest of the community who might not be watching cfe-commits -- that's a very fast turnaround and very little notification for such a significant change.
>>
>> Yeah this is on me. Honestly I didn't expect it to be that much of a problem but rather the toolchain requirement we posted as part of it would be the big hurdle where bot owners would have to upgrade to get the right versions. But lesson learned  and we should add some more delays in the policy here: https://llvm.org/docs/DeveloperPolicy.html#id23 and cover the C++ standards upgrade.
>
> Two points I want to add that I think would've been useful as well -
>
> 1. In addition to the toolchain soft errors, add a version check + #warning to some llvm header. This would be useful as it is more visible than the CMake warning and it could show up for cases where LLVM is used as a library+headers and not built from sources.
> 2. Delay actual usage of new language features until after the next release. Currently I see people pushing lots of cleanup commits that could hurt bug backports. It also has the benefit of making the transition more gradual.

Strong +1 to point #2 especially. This is something we could have plausibly reverted to work through the kinks rather than doing that work live and while under duress, but it became implausible pretty quickly because everyone started landing their C++17 NFC changes. Those kinds of changes almost always can wait until after we've validated that the switch has gone smoothly.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130689