[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