[PATCH] D77551: [analyzer] Fix NSErrorChecker false positives

Kristóf Umann via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 6 10:18:51 PDT 2020


Szelethus added inline comments.


================
Comment at: clang/test/Analysis/SpecialFunctionsCFError.cpp:2
+// RUN: %clang_analyze_cc1 -analyzer-checker=core,osx.coreFoundation.CFError \
+// RUN:   -analyzer-store=region -verify %s
+
----------------
NoQ wrote:
> `-analyzer-store=region` is redundant; it's the default and the only possible value for this option. We should remove it from other tests as well.
The default lit command is a bit silly. What does `-setup-analyzer` or something like that even do?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77551/new/

https://reviews.llvm.org/D77551





More information about the cfe-commits mailing list