diff options
| author | Dan Gohman <gohman@apple.com> | 2011-09-29 22:27:34 +0000 |
|---|---|---|
| committer | Dan Gohman <gohman@apple.com> | 2011-09-29 22:27:34 +0000 |
| commit | 27e0666725c6558574cb7a55f8c91e7532e09548 (patch) | |
| tree | edade80c839f45c971878899d08c9710caab95f1 /lib/Transforms | |
| parent | 597fece886fb4551d1f26ec7749d86e89970d8c2 (diff) | |
When eliminating unnecessary retain+autorelease on return values,
handle the case where the retain is in a different basic block.
rdar://10210274.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@140815 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms')
| -rw-r--r-- | lib/Transforms/Scalar/ObjCARC.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/lib/Transforms/Scalar/ObjCARC.cpp b/lib/Transforms/Scalar/ObjCARC.cpp index 29d0fe6a2e..da74e9c3ec 100644 --- a/lib/Transforms/Scalar/ObjCARC.cpp +++ b/lib/Transforms/Scalar/ObjCARC.cpp @@ -3198,7 +3198,8 @@ void ObjCARCOpt::OptimizeReturns(Function &F) { // Check that there is nothing that can affect the reference // count between the retain and the call. - FindDependencies(CanChangeRetainCount, Arg, BB, Retain, + // Note that Retain need not be in BB. + FindDependencies(CanChangeRetainCount, Arg, Retain->getParent(), Retain, DependingInstructions, Visited, PA); if (DependingInstructions.size() != 1) goto next_block; |