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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to