[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