[PATCH] D15559: [LoopUnrollRuntime] Do unroll when IV's setp is one or minus one

Sanjoy Das via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 22 01:12:10 PST 2015

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

I'd rather have this sort of logic live in `SCEVExpander::isHighCostExpansion` than be scattered throughout the codebase.  I think the "increment by one" case is covered by `findExistingExpansion` already; and the "decrement by one" case can be added there as well by teaching it to look at the initial values for the induction phi nodes.


More information about the llvm-commits mailing list