diff options
| author | Matthijs Kooijman <matthijs@stdin.nl> | 2008-07-16 10:47:35 +0000 |
|---|---|---|
| committer | Matthijs Kooijman <matthijs@stdin.nl> | 2008-07-16 10:47:35 +0000 |
| commit | b85cc9db19abb46b4ebf1e917320093c3ac5848f (patch) | |
| tree | 30c6ba03ce2deea3f8b02ad4af5a849375f12be7 /llvm/lib/Analysis/ValueTracking.cpp | |
| parent | a0b1ab3c0bdd8982c86beaa406b2a6a61993a0fb (diff) | |
| download | bcm5719-llvm-b85cc9db19abb46b4ebf1e917320093c3ac5848f.tar.gz bcm5719-llvm-b85cc9db19abb46b4ebf1e917320093c3ac5848f.zip | |
Don't use ++idx_begin when I actually mean idx_begin + 1, especially since we
also use *idx_begin in the same expression, giving unpredictable results.
This fixes this bug: http://lists.cs.uiuc.edu/pipermail/llvmdev/2008-July/015877.html
llvm-svn: 53670
Diffstat (limited to 'llvm/lib/Analysis/ValueTracking.cpp')
| -rw-r--r-- | llvm/lib/Analysis/ValueTracking.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Analysis/ValueTracking.cpp b/llvm/lib/Analysis/ValueTracking.cpp index e7e291fb928..e35f0d0fced 100644 --- a/llvm/lib/Analysis/ValueTracking.cpp +++ b/llvm/lib/Analysis/ValueTracking.cpp @@ -864,7 +864,7 @@ Value *llvm::FindInsertedValue(Value *V, const unsigned *idx_begin, else if (Constant *C = dyn_cast<Constant>(V)) { if (isa<ConstantArray>(C) || isa<ConstantStruct>(C)) // Recursively process this constant - return FindInsertedValue(C->getOperand(*idx_begin), ++idx_begin, idx_end, + return FindInsertedValue(C->getOperand(*idx_begin), idx_begin + 1, idx_end, InsertBefore); } else if (InsertValueInst *I = dyn_cast<InsertValueInst>(V)) { // Loop the indices for the insertvalue instruction in parallel with the |
