diff options
author | Chandler Carruth <chandlerc@gmail.com> | 2012-10-02 22:46:45 +0000 |
---|---|---|
committer | Chandler Carruth <chandlerc@gmail.com> | 2012-10-02 22:46:45 +0000 |
commit | 5c5b3cf5b8af06b8e9347f3f45e8c67438ffd446 (patch) | |
tree | 8d0f5ecadba9f4350cceb812320fd71d8ddb0971 /lib/Transforms/Scalar/SROA.cpp | |
parent | 5d37976090df34f003e5128e39593b763be0ca71 (diff) | |
download | llvm-5c5b3cf5b8af06b8e9347f3f45e8c67438ffd446.tar.gz llvm-5c5b3cf5b8af06b8e9347f3f45e8c67438ffd446.tar.bz2 llvm-5c5b3cf5b8af06b8e9347f3f45e8c67438ffd446.tar.xz |
Teach the new SROA to handle cases where an alloca that has already been
scheduled for processing on the worklist eventually gets deleted while
we are processing another alloca, fixing the original test case in
PR13990.
To facilitate this, add a remove_if helper to the SetVector abstraction.
It's not easy to use the standard abstractions for this because of the
specifics of SetVectors types and implementation.
Finally, a nice small test case is included. Thanks to Benjamin for the
fantastic reduced test case here! All I had to do was delete some empty
basic blocks!
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@165065 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms/Scalar/SROA.cpp')
-rw-r--r-- | lib/Transforms/Scalar/SROA.cpp | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/lib/Transforms/Scalar/SROA.cpp b/lib/Transforms/Scalar/SROA.cpp index 61d49faf38..58c3bc0f20 100644 --- a/lib/Transforms/Scalar/SROA.cpp +++ b/lib/Transforms/Scalar/SROA.cpp @@ -3302,7 +3302,11 @@ bool SROA::runOnFunction(Function &F) { while (!Worklist.empty()) { Changed |= runOnAlloca(*Worklist.pop_back_val()); deleteDeadInstructions(DeletedAllocas); + + // Remove the deleted allocas from various lists so that we don't try to + // continue processing them. if (!DeletedAllocas.empty()) { + Worklist.remove_if(IsAllocaInSet(DeletedAllocas)); PromotableAllocas.erase(std::remove_if(PromotableAllocas.begin(), PromotableAllocas.end(), IsAllocaInSet(DeletedAllocas)), |