diff options
| author | Dan Gohman <gohman@apple.com> | 2009-03-09 21:04:19 +0000 |
|---|---|---|
| committer | Dan Gohman <gohman@apple.com> | 2009-03-09 21:04:19 +0000 |
| commit | 53f2ae268ab0e9201c0c50c28a27d451497a0371 (patch) | |
| tree | 1ad0aeb4a5d214a3ef772ff8153b3f56fc64b61a | |
| parent | 21e7722868378f67974e648ab21d0e3c69a0e379 (diff) | |
Don't use an induction variable type as a memory access type.
Use VoidTy instead, to be properly conservative.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@66463 91177308-0d34-0410-b5e6-96231b3b80d8
| -rw-r--r-- | lib/Transforms/Scalar/LoopStrengthReduce.cpp | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/lib/Transforms/Scalar/LoopStrengthReduce.cpp b/lib/Transforms/Scalar/LoopStrengthReduce.cpp index c465a076e2..b64c689df5 100644 --- a/lib/Transforms/Scalar/LoopStrengthReduce.cpp +++ b/lib/Transforms/Scalar/LoopStrengthReduce.cpp @@ -1774,7 +1774,7 @@ LoopStrengthReduce::PrepareToStrengthReduceFromSmallerStride( } static bool IsImmFoldedIntoAddrMode(GlobalValue *GV, int64_t Offset, - const Type *ReplacedTy, + const Type *AccessTy, std::vector<BasedUser> &UsersToProcess, const TargetLowering *TLI) { SmallVector<Instruction*, 16> AddrModeInsts; @@ -1783,7 +1783,7 @@ static bool IsImmFoldedIntoAddrMode(GlobalValue *GV, int64_t Offset, continue; ExtAddrMode AddrMode = AddressingModeMatcher::Match(UsersToProcess[i].OperandValToReplace, - ReplacedTy, UsersToProcess[i].Inst, + AccessTy, UsersToProcess[i].Inst, AddrModeInsts, *TLI); if (GV && GV != AddrMode.BaseGV) return false; @@ -1858,7 +1858,9 @@ void LoopStrengthReduce::StrengthReduceStridedIVUsers(const SCEVHandle &Stride, if (SCEVConstant *SC = dyn_cast<SCEVConstant>(Imm)) Offset = SC->getValue()->getSExtValue(); if (GV || Offset) - DoSink = IsImmFoldedIntoAddrMode(GV, Offset, ReplacedTy, + // Pass VoidTy as the AccessTy to be conservative, because + // there could be multiple access types among all the uses. + DoSink = IsImmFoldedIntoAddrMode(GV, Offset, Type::VoidTy, UsersToProcess, TLI); if (DoSink) { |