diff options
author | John McCall <rjmccall@apple.com> | 2011-07-07 06:58:02 +0000 |
---|---|---|
committer | John McCall <rjmccall@apple.com> | 2011-07-07 06:58:02 +0000 |
commit | 7e5e5f4cc36fe50f46ad76dca7a266434c94f475 (patch) | |
tree | 58c8d913ddf2c68a1558a3a0792b3288ee0d04a1 /lib/ARCMigrate/TransformActions.cpp | |
parent | 72ac120023abb73f3ff9386e193fed55fa9a96e2 (diff) |
In ARC, reclaim all return values of retainable type, not just those
where we have an immediate need of a retained value.
As an exception, don't do this when the call is made as the immediate
operand of a __bridge retain. This is more in the way of a workaround
than an actual guarantee, so it's acceptable to be brittle here.
rdar://problem/9504800
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@134605 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/ARCMigrate/TransformActions.cpp')
-rw-r--r-- | lib/ARCMigrate/TransformActions.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/ARCMigrate/TransformActions.cpp b/lib/ARCMigrate/TransformActions.cpp index 45a3d473e6..2ae979896a 100644 --- a/lib/ARCMigrate/TransformActions.cpp +++ b/lib/ARCMigrate/TransformActions.cpp @@ -313,7 +313,7 @@ void TransformActionsImpl::removeStmt(Stmt *S) { assert(IsInTransaction && "Actions only allowed during a transaction"); ActionData data; data.Kind = Act_RemoveStmt; - data.S = S; + data.S = S->IgnoreImplicit(); // important for uniquing CachedActions.push_back(data); } |