[PATCH] D22817: Utility to print all the basic blocks of a loop.

Sebastian Pop via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 26 22:12:11 PDT 2016


sebpop added a comment.

In https://reviews.llvm.org/D22817#496596, @sanjoy wrote:

> LGTM with comments inline.
>
> However: if it isn't too much extra work, can I ask you to please land
>  the `Verbose` flag in a separate change from the `isLoopLatch` change?
>  If that's too much extra work, don't bother, since these are low risk
>  changes.


Split in two patches and addressed all last comments:
https://reviews.llvm.org/rL276837
https://reviews.llvm.org/rL276838

Thanks Sanjoy for your review.


Repository:
  rL LLVM

https://reviews.llvm.org/D22817





More information about the llvm-commits mailing list