erichkeane wrote:

> > Why is this a new review? What changed from the last one I looked at?
> 
> Ah, right -- this patch is actually derived from the previous one, and it is 
> simpler. That older patch became a bit outdated since some of its changes 
> were split out and already landed in main. I found it easier to start fresh 
> with a new patch rather than rebasing the old one.
> 
> That said, if you’d prefer to stick with the old review (to preserve the 
> initial comments there), I can force-push this patch to the old review 
> instead.

Ah, not a problem, the new one is fine.  I was just very thrown/wondering why I 
was reviewing something so similar again.

https://github.com/llvm/llvm-project/pull/147292
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to