6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-01-03 13:31:05 +00:00
Code Issues 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!


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@14358 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Chris Lattner 2004-06-24 06:49:18 +00:00
parent 7548a540f7
commit a25502acd7
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! if (PN->getType()->isInteger()) { // FIXME: when we have fast-math, enable!
SCEVHandle SCEV = SE->getSCEV(PN); SCEVHandle SCEV = SE->getSCEV(PN);
if (SCEV->hasComputableLoopEvolution(L)) 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)); IndVars.push_back(std::make_pair(PN, SCEV));
} }