RPCS3/llvm-mirror
1
0
Fork 0
mirror of https://github.com/RPCS3/llvm-mirror.git synced 2024-12-11 13:37:07 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Two fixes. First, stop using the ugly shouldSubstituteIndVar method.

Browse Source 
Second, disable substitution of quadratic addrec expressions to avoid putting
multiplies in loops!

llvm-svn: 14358
This commit is contained in:
Chris Lattner 2004-06-24 06:49:18 +00:00
parent 236ef1542c
commit b4df848a58
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
lib/Transforms/Scalar/IndVarSimplify.cpp
View File

@ -602,7 +602,13 @@ void IndVarSimplify::runOnLoop(Loop *L) {
if (PN->getType()->isInteger()) { // FIXME: when we have fast-math, enable!
SCEVHandle SCEV = SE->getSCEV(PN);
if (SCEV->hasComputableLoopEvolution(L))
if (SE->shouldSubstituteIndVar(SCEV)) // HACK!
// FIXME: Without a strength reduction pass, it is an extremely bad idea
// to indvar substitute anything more complex than a linear induction
// variable. Doing so will put expensive multiply instructions inside
// of the loop. For now just disable indvar subst on anything more
// complex than a linear addrec.
if (!isa<SCEVAddRecExpr>(SCEV) ||
cast<SCEVAddRecExpr>(SCEV)->getNumOperands() < 3)
IndVars.push_back(std::make_pair(PN, SCEV));
}