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

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 6 10:51:26 PDT 2020


NoQ 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
+
----------------
vsavchenko wrote:
> Szelethus wrote:
> > 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?
> Should I remove it from here in this commit or we should clean up all tests at once?
> The default lit command is a bit silly. What does `-setup-analyzer` or something like that even do?

```lang=c
  1029   if (PPOpts.SetUpStaticAnalyzer)
  1030     Builder.defineMacro("__clang_analyzer__");
```

:)

> Should I remove it from here in this commit or we should clean up all tests at once?
I'll remove when committing, thx!


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