[PATCH] D20765: Unroll once when -unroll-count is set

Michael Zolotukhin via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 2 17:14:28 PDT 2016


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

LGTM with a minor remark below.

Michael


================
Comment at: test/Transforms/LoopUnroll/unroll-count.ll:2
@@ +1,3 @@
+; RUN: opt < %s -S -loop-unroll -unroll-count=2 | FileCheck %s
+; Checks that unroll with count set by user occur only once.
+;
----------------
The test doesn't check that unrolling occurs only once, it only checks if the metadata is set after unrolling - this is totally fine, I just think the comment is slightly off.


Repository:
  rL LLVM

http://reviews.llvm.org/D20765





More information about the llvm-commits mailing list