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

[Michael Zolotukhin via llvm-commits]

mzolotukhin requested changes to this revision.
mzolotukhin added a comment.
This revision now requires changes to proceed.

Hi Evgeny,

The change looks good, but I'd like the test to be changed. We don't need to run entire O2 pipeline for it. If you're trying to catch some behavior of the second invocation of loop-unroll, please just add IR from right before that as another test.

Thanks,
Michael


================
Comment at: test/Transforms/LoopUnroll/runtime-count.ll:1
@@ +1,2 @@
+; RUN: opt < %s -S -O2 -unroll-count=2 | FileCheck %s
+; Loop should be unrolled 2 times (not 4)
----------------
Please run only `-loop-unroll`, not entire `-O2`.


Repository:
  rL LLVM

http://reviews.llvm.org/D20765