[PATCH] D24451: [LoopUnroller] Replace UnrollingPreferences::Force with ForceMaxCount + SystemZ getUnrollingPreferences().

[Evgeny Stupachenko via llvm-commits]

evstupac added a comment.

> So the idea here is to force-unroll just enough to make the loop bigger than 6 instructions at least, but not more.


Then Threshold is better for your case. You should bound unrolling by 6. And first try to bound Runtime unroll (and therefore Force unroll as well). Only if this does not help create new Threshold for forced unroll.

With your current changes loop that has 150 iterations (UP.Threshold) after unroll is ok... that could unroll by 2 of 50 instructions loop.


https://reviews.llvm.org/D24451