diff options
| author | Anna Zaks <ganna@apple.com> | 2012-05-07 17:47:09 +0000 |
|---|---|---|
| committer | Anna Zaks <ganna@apple.com> | 2012-05-07 17:47:09 +0000 |
| commit | 41a669a30074dcc221ba199e5dde484cc33adba1 (patch) | |
| tree | ae2627f0dd51f93d5ce5f2279f6c4adace9e2934 /lib/StaticAnalyzer/Checkers/RetainCountChecker.cpp | |
| parent | fcf7f9fdcbec10a1d9640ad6960c7b6fe7a19fee (diff) | |
[analyzer]Fixup r156215: use StopTracking summary instead of ScratchArgs
As per Jordy's and Ted's comment, use the default StopTracking summary
instead of adding all arguments to ScratchArs.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@156310 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/StaticAnalyzer/Checkers/RetainCountChecker.cpp')
| -rw-r--r-- | lib/StaticAnalyzer/Checkers/RetainCountChecker.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/lib/StaticAnalyzer/Checkers/RetainCountChecker.cpp b/lib/StaticAnalyzer/Checkers/RetainCountChecker.cpp index 2cf9353771..325a7657e6 100644 --- a/lib/StaticAnalyzer/Checkers/RetainCountChecker.cpp +++ b/lib/StaticAnalyzer/Checkers/RetainCountChecker.cpp @@ -1005,10 +1005,8 @@ RetainSummaryManager::getSummary(const FunctionDecl *FD, ScratchArgs = AF.add(ScratchArgs, 2, StopTracking); S = getPersistentSummary(RetEffect::MakeNoRet(), DoNothing, DoNothing); } else if (CME && CME->hasNonZeroCallbackArg()) { - // Allow objects to escape throug callbacks. radar://10973977 - for (unsigned I = 0; I < CME->getNumArgs(); ++I) - ScratchArgs = AF.add(ScratchArgs, I, StopTracking); - S = getPersistentSummary(RetEffect::MakeNoRet(), DoNothing, DoNothing); + // Allow objects to escape through callbacks. radar://10973977 + S = getPersistentStopSummary(); } // Did we get a summary? |