[PATCH] D134231: [clang][C++20] p0960 Allow initializing aggregates from a parenthesized list of values

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 7 06:46:44 PST 2022


aaron.ballman added a comment.

In D134231#3910882 <https://reviews.llvm.org/D134231#3910882>, @ecatmur wrote:

> In D134231#3803380 <https://reviews.llvm.org/D134231#3803380>, @shafik wrote:
>
>> Hello Ed, thank you for picking up this work.
>>
>> I liked the approach in the first PR, did you consider reaching out to the author is asking if it was ok to commandeer the work to allow it to move forward?
>
> Hi Shafik,
>
> No, I didn't consider that - sorry! I felt that having done this work independently it was worth presenting separately. Sorry if I wasted anyone's time and happy to see that Alan has picked up the other PR.

No worries at all, I think it's great that we had a few independent approaches to this, so definitely not a waste of time IMO.

> I'll close this as superseded if I can work out how.

If you go to the "Add Action" dropdown, there should be an "Abandon Revision" option (or something along those lines).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134231



More information about the cfe-commits mailing list