[PATCH] D18670: LoopUnroll: some small fixes/tweaks to make it more useful for partial unrolling

Michael Zolotukhin via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 4 12:44:53 PDT 2016


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

LGTM!

Michael


================
Comment at: test/Transforms/LoopUnroll/partial-unroll-maxcount.ll:19-22
@@ +18,5 @@
+
+; CHECK:      add
+; CHECK-NEXT: add
+; CHECK-NEXT: add
+; CHECK-NEXT: icmp
----------------
I think in most of the tests we have CHECK-lines before the test. Could you please move it there and add CHECK-LABEL in the beginning (in case we add more tests in future)?


Repository:
  rL LLVM

http://reviews.llvm.org/D18670





More information about the llvm-commits mailing list