[cfe-commits] r156215 - in /cfe/trunk: lib/StaticAnalyzer/Checkers/RetainCountChecker.cpp test/Analysis/retain-release.m

Anna Zaks ganna at apple.com
Mon May 7 10:48:47 PDT 2012


In r156310.

Thanks for the review,
Anna.
On May 4, 2012, at 10:14 PM, Ted Kremenek wrote:

> On May 4, 2012, at 8:56 PM, Jordy Rose <jediknil at belkadan.com> wrote:
> 
>> One of the arguments to getPersistentSummary is the default effect on the arguments, so you shouldn't have to use ScratchArgs to do this. In fact, since summaries are uniqued, not using ScratchArgs could actually save memory in callback-heavy code.
> 
> To that point, I think we can just use the "StopTracking" summary here, which is precomputed for this purpose.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120507/7a6e2681/attachment.html>


More information about the cfe-commits mailing list