maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[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.

llvm-svn: 156310
This commit is contained in:
Anna Zaks 2012-05-07 17:47:09 +00:00
parent 0ae9efeb1a
commit 41e7ab8f81
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
clang/lib/StaticAnalyzer/Checkers/RetainCountChecker.cpp
View File

@ -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?