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

[escha via llvm-commits]

escha added a comment.

Going by the code, Count is emphatically not what I want:

  if (!AllowPartial && !CountSetExplicitly) {
    DEBUG(dbgs() << "  will not try to unroll partially because "
                 << "-unroll-allow-partial not given\n");
    return false;
  }
  if (!AllowRuntime && !CountSetExplicitly) {
    DEBUG(dbgs() << "  will not try to unroll loop with runtime trip count "
                 << "-unroll-runtime not given\n");
    return false;
  }

It forces an unroll of Count, even if you haven't enabled Runtime unrolling and don't want it.


Repository:
  rL LLVM

http://reviews.llvm.org/D18670