[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