[PATCH] D61839: [analyzer][WIP] Hide developer-only checker/package options by default

Kristóf Umann via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun May 12 16:44:09 PDT 2019


Szelethus created this revision.
Szelethus added reviewers: NoQ, dcoughlin, xazax.hun, rnkovacs, Charusso, a_sidorin, baloghadamsoftware.
Szelethus added a project: clang.
Herald added subscribers: cfe-commits, gamesh411, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, szepet, whisperity.
Szelethus added a parent revision: D57858: [analyzer] Add a new frontend flag to display all checker options.
Szelethus edited the summary of this revision.

Followup to D60925 <https://reviews.llvm.org/D60925>, this time, for checker options. Non-checker configurations should receive the same treatment eventually.

The patch is marked WIP, because I am not really confident about osx checkers, and I can see valid uses for the rest of the options, but I might not be strict enough on this. Also, @NoQ suggested that hidden checker's options should be hidden by default, but I'm kind of hesitant about it, as described in D57858#1499414 <https://reviews.llvm.org/D57858#1499414>.


Repository:
  rC Clang

https://reviews.llvm.org/D61839

Files:
  include/clang/Driver/CC1Options.td
  include/clang/StaticAnalyzer/Checkers/CheckerBase.td
  include/clang/StaticAnalyzer/Checkers/Checkers.td
  include/clang/StaticAnalyzer/Core/AnalyzerOptions.h
  include/clang/StaticAnalyzer/Frontend/CheckerRegistry.h
  lib/Frontend/CompilerInvocation.cpp
  lib/FrontendTool/ExecuteCompilerInvocation.cpp
  lib/StaticAnalyzer/Frontend/CheckerRegistry.cpp
  test/Analysis/analyzer-checker-option-help.c
  utils/TableGen/ClangSACheckersEmitter.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D61839.199190.patch
Type: text/x-patch
Size: 15062 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190512/e14112fd/attachment-0001.bin>


More information about the cfe-commits mailing list