[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.


http://reviews.llvm.org/D15559





More information about the llvm-commits mailing list