From 957fd1704a21e91343dfe3a850c512c40b49de23 Mon Sep 17 00:00:00 2001 From: Dan Gohman Date: Mon, 29 Oct 2007 19:23:53 +0000 Subject: [PATCH] Avoid calling ValidStride when not all uses are addresses. llvm-svn: 43460 --- lib/Transforms/Scalar/LoopStrengthReduce.cpp | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/lib/Transforms/Scalar/LoopStrengthReduce.cpp b/lib/Transforms/Scalar/LoopStrengthReduce.cpp index 9c955a99b0e..e3845081399 100644 --- a/lib/Transforms/Scalar/LoopStrengthReduce.cpp +++ b/lib/Transforms/Scalar/LoopStrengthReduce.cpp @@ -1013,15 +1013,14 @@ unsigned LoopStrengthReduce::CheckForIVReuse(bool HasBaseReg, (unsigned(abs(SInt)) < SSInt || (SInt % SSInt) != 0)) continue; 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 // 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 - // anyway. - if (ValidStride(HasBaseReg, Scale, UsersToProcess)) + // anyway. If the scale is 1, then we don't need to worry about folding + // multiplications. + if (Scale == 1 || + (AllUsesAreAddresses && + ValidStride(HasBaseReg, Scale, UsersToProcess))) for (std::vector::iterator II = SI->second.IVs.begin(), IE = SI->second.IVs.end(); II != IE; ++II) // FIXME: Only handle base == 0 for now.