From aa34331e7832dc1139231626516e9587eeecb0ce 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. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@43460 91177308-0d34-0410-b5e6-96231b3b80d8 --- 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.