6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-02-13 10:32:06 +00:00
Code Issues Projects Releases Wiki Activity

Avoid calling ValidStride when not all uses are addresses.

Browse Source 
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@43460 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Dan Gohman 2007-10-29 19:23:53 +00:00
parent 8308a48536
commit aa34331e78
1 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
lib/Transforms/Scalar/LoopStrengthReduce.cpp
View File

@ -1013,15 +1013,14 @@ unsigned LoopStrengthReduce::CheckForIVReuse(bool HasBaseReg,
(unsigned(abs(SInt)) < SSInt || (SInt % SSInt) != 0)) (unsigned(abs(SInt)) < SSInt || (SInt % SSInt) != 0))
continue; continue;
int64_t Scale = SInt / SSInt; int64_t Scale = SInt / SSInt;
// When scale is 1, we don't need to worry about whether the
// multiplication can be folded into the addressing mode.
if (!AllUsesAreAddresses && Scale != 1)
continue;
// Check that this stride is valid for all the types used for loads and // Check that this stride is valid for all the types used for loads and
// stores; if it can be used for some and not others, we might as well use // stores; if it can be used for some and not others, we might as well use
// the original stride everywhere, since we have to create the IV for it // the original stride everywhere, since we have to create the IV for it
// anyway. // anyway. If the scale is 1, then we don't need to worry about folding
if (ValidStride(HasBaseReg, Scale, UsersToProcess)) // multiplications.
if (Scale == 1 ||
(AllUsesAreAddresses &&
ValidStride(HasBaseReg, Scale, UsersToProcess)))
for (std::vector<IVExpr>::iterator II = SI->second.IVs.begin(), for (std::vector<IVExpr>::iterator II = SI->second.IVs.begin(),
IE = SI->second.IVs.end(); II != IE; ++II) IE = SI->second.IVs.end(); II != IE; ++II)
// FIXME: Only handle base == 0 for now. // FIXME: Only handle base == 0 for now.