diff options
| author | Chris Lattner <sabre@nondot.org> | 2011-04-09 05:51:34 +0000 |
|---|---|---|
| committer | Chris Lattner <sabre@nondot.org> | 2011-04-09 05:51:34 +0000 |
| commit | c9049a9ea8993e64b58c206fed6609a1415519f2 (patch) | |
| tree | 50d6b1ae5ab073afdba47ab43e753f578dc02554 /include/llvm/Analysis/AliasSetTracker.h | |
| parent | f4afdfc501b7185d24a0ef184fe3d0c0bbe22e0c (diff) | |
fix a potentially serious bug in AliasSet::removeCallSite
where we shrunk the list without updating the end iterator.
By inspection, from PR9639.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@129190 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'include/llvm/Analysis/AliasSetTracker.h')
| -rw-r--r-- | include/llvm/Analysis/AliasSetTracker.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/include/llvm/Analysis/AliasSetTracker.h b/include/llvm/Analysis/AliasSetTracker.h index e844d10dda..4efa3963b6 100644 --- a/include/llvm/Analysis/AliasSetTracker.h +++ b/include/llvm/Analysis/AliasSetTracker.h @@ -259,6 +259,7 @@ private: if (CallSites[i] == CS.getInstruction()) { CallSites[i] = CallSites.back(); CallSites.pop_back(); + --i; --e; // Revisit the moved entry. } } void setVolatile() { Volatile = true; } |