[PATCH] D124351: [Clang][WIP] Implement Change scope of lambda trailing-return-type - Take 2

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 21 05:37:54 PST 2022


aaron.ballman added a comment.

In D124351#3940890 <https://reviews.llvm.org/D124351#3940890>, @cor3ntin wrote:

> In D124351#3940888 <https://reviews.llvm.org/D124351#3940888>, @aaron.ballman wrote:
>
>> In D124351#3940786 <https://reviews.llvm.org/D124351#3940786>, @cor3ntin wrote:
>>
>>> Now that Kona is over, I'm hoping to get back to that in the coming weeks. Rebasing will be... fun.
>>> I want to make sure we are all okay making that a DR following WG21 guidance, given that not making it a DR would have potentially large impact on the PR.
>>> Thanks!
>>
>> To me, it depends on how much existing code the DR breaks in practice. If we don't expect it to break a significant body of code (or if the code that breaks will be materially improved as a result of the required changes), then we should implement it as a DR as far back as we can go.
>
> We expect very little, if any, breakage

Then I think we should approach it as a DR and we can react if we get further feedback.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124351



More information about the cfe-commits mailing list