aboutsummaryrefslogtreecommitdiffstats
path: root/lib/Transforms
diff options
context:
space:
mode:
authorJuergen Ributzka <juergen@apple.com>2013-10-24 05:29:56 +0000
committerJuergen Ributzka <juergen@apple.com>2013-10-24 05:29:56 +0000
commitd084153a8fd8ee058fd4c39d4f15d3f988566d95 (patch)
tree1353ec643914e9de2d02a9d9eb3d6500b1bfaabb /lib/Transforms
parentcbbd20879e8acf35d3326a2459e17a298b5f5d15 (diff)
downloadexternal_llvm-d084153a8fd8ee058fd4c39d4f15d3f988566d95.zip
external_llvm-d084153a8fd8ee058fd4c39d4f15d3f988566d95.tar.gz
external_llvm-d084153a8fd8ee058fd4c39d4f15d3f988566d95.tar.bz2
Fix a bug in LinearFunctionTestReplace that created invalid loop exit checks.
Reviewed by Andy git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@193303 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms')
-rw-r--r--lib/Transforms/Scalar/IndVarSimplify.cpp8
1 files changed, 7 insertions, 1 deletions
diff --git a/lib/Transforms/Scalar/IndVarSimplify.cpp b/lib/Transforms/Scalar/IndVarSimplify.cpp
index 7785855..cfd8db0 100644
--- a/lib/Transforms/Scalar/IndVarSimplify.cpp
+++ b/lib/Transforms/Scalar/IndVarSimplify.cpp
@@ -1479,8 +1479,14 @@ static Value *genLoopLimit(PHINode *IndVar, const SCEV *IVCount, Loop *L,
if (IndVar->getType()->isPointerTy()
&& !IVCount->getType()->isPointerTy()) {
+ // IVOffset will be the new GEP offset that is interpreted by GEP as a
+ // signed value. IVCount on the other hand represents the loop trip count,
+ // which is an unsigned value. FindLoopCounter only allows induction
+ // variables that have a positive unit stride of one. This means we don't
+ // have to handle the case of negative offsets (yet) and just need to zero
+ // extend IVCount.
Type *OfsTy = SE->getEffectiveSCEVType(IVInit->getType());
- const SCEV *IVOffset = SE->getTruncateOrSignExtend(IVCount, OfsTy);
+ const SCEV *IVOffset = SE->getTruncateOrZeroExtend(IVCount, OfsTy);
// Expand the code for the iteration count.
assert(SE->isLoopInvariant(IVOffset, L) &&