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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to