hokein 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. https://github.com/llvm/llvm-project/pull/147292 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
