diff options
| author | Benjamin Kramer <benny.kra@googlemail.com> | 2012-09-09 16:44:05 +0000 |
|---|---|---|
| committer | Benjamin Kramer <benny.kra@googlemail.com> | 2012-09-09 16:44:05 +0000 |
| commit | 3be7584f40cc822252e613af0c42dcb17e6a5d05 (patch) | |
| tree | fe6862d9b0c4543ea8c0e4279afa3ce3ca9ea4ef /lib/Transforms/Scalar/DeadStoreElimination.cpp | |
| parent | 11b450589978a39e59b77cd074dcda9d5697f174 (diff) | |
| download | llvm-3be7584f40cc822252e613af0c42dcb17e6a5d05.tar.gz llvm-3be7584f40cc822252e613af0c42dcb17e6a5d05.tar.bz2 llvm-3be7584f40cc822252e613af0c42dcb17e6a5d05.tar.xz | |
DSE: Poking holes into a SetVector is expensive, avoid it if possible.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@163480 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms/Scalar/DeadStoreElimination.cpp')
| -rw-r--r-- | lib/Transforms/Scalar/DeadStoreElimination.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/lib/Transforms/Scalar/DeadStoreElimination.cpp b/lib/Transforms/Scalar/DeadStoreElimination.cpp index 25a1dd770a..1ff4329c84 100644 --- a/lib/Transforms/Scalar/DeadStoreElimination.cpp +++ b/lib/Transforms/Scalar/DeadStoreElimination.cpp @@ -775,15 +775,15 @@ bool DSE::handleEndBlock(BasicBlock &BB) { LiveAllocas.push_back(*I); } - for (SmallVector<Value*, 8>::iterator I = LiveAllocas.begin(), - E = LiveAllocas.end(); I != E; ++I) - DeadStackObjects.remove(*I); - // If all of the allocas were clobbered by the call then we're not going // to find anything else to process. - if (DeadStackObjects.empty()) + if (DeadStackObjects.size() == LiveAllocas.size()) break; + for (SmallVector<Value*, 8>::iterator I = LiveAllocas.begin(), + E = LiveAllocas.end(); I != E; ++I) + DeadStackObjects.remove(*I); + continue; } |