[PATCH] D32621: TargetLowering: Add finalizeLowering() function; NFC

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 28 12:34:44 PDT 2017


rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D32621#741133, @MatzeB wrote:

> In https://reviews.llvm.org/D32621#741035, @rnk wrote:
>
> > Oops, I went straight to the diff and missed Matt's comment.
>
>
> Does that mean you want to see Matts comment addressed (I think we both agreed to do that in a separate patch), or just need more time for review?


No, I looked at the other patch and that seems fine. Looks good!


Repository:
  rL LLVM

https://reviews.llvm.org/D32621





More information about the llvm-commits mailing list