aaron.ballman added a comment.

In D126937#3555731 <https://reviews.llvm.org/D126937#3555731>, @aaron.ballman 
wrote:

> In D126937#3555243 <https://reviews.llvm.org/D126937#3555243>, @browneee 
> wrote:
>
>> I was also looking into fixing this: https://reviews.llvm.org/D126944
>>
>> I'm not yet sure if my changes are correct.
>
> I did a review on https://reviews.llvm.org/D126944, but I'll let the two of 
> you decide which review to move forward with rather than review them both in 
> tandem. :-)

FYI, I've accepted that other review (we needed to get the leak fixed ASAP).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126937

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

Reply via email to