[PATCH] D39438: [analyzer] Diagnose stack leaks via block captures

Alexander Shaposhnikov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 1 18:36:21 PDT 2017


alexshap added inline comments.


================
Comment at: test/Analysis/stack-async-leak.m:1
+// RUN: %clang_analyze_cc1 -triple x86_64-apple-darwin10 -analyzer-checker=core -fblocks -fobjc-arc -verify %s
+
----------------
dcoughlin wrote:
> Can you add an additional run line testing the expected behavior when ARC is not enabled? You can pass a -DARC_ENABLED=1 or similar to distinguish between the two (for example, to test to make sure that a diagnostic is emitted under MRR but not ARC).
i've tried this, it somehow gets more complicated to read / change the tests - decided to create a separate file.


Repository:
  rL LLVM

https://reviews.llvm.org/D39438





More information about the cfe-commits mailing list