[PATCH] D39438: [analyzer] Diagnose stack leaks via block captures
Alexander Shaposhnikov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Nov 2 20:08:45 PDT 2017
alexshap updated this revision to Diff 121431.
alexshap added a comment.
1. Add no-warning comments
2. Switch to using a cached IdentifierInfo (similar to how it's done in CheckObjCDealloc.cpp)
3. Rerun the all the tests - they are fine.
@dcoughlin - many thanks for the code review,
yeah, i will wait for the feedback from NoQ@ as well.
Repository:
rL LLVM
https://reviews.llvm.org/D39438
Files:
lib/StaticAnalyzer/Checkers/StackAddrEscapeChecker.cpp
test/Analysis/stack-capture-leak-arc.mm
test/Analysis/stack-capture-leak-no-arc.mm
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D39438.121431.patch
Type: text/x-patch
Size: 22318 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20171103/5907848e/attachment-0001.bin>
More information about the cfe-commits
mailing list