[PATCH] D57922: [analyzer] Insert checker options into AnalyzerOption::ConfigTable

Kristóf Umann via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 7 13:22:47 PST 2019


Szelethus created this revision.
Szelethus added reviewers: NoQ, george.karpenkov, xazax.hun, rnkovacs, baloghadamsoftware.
Herald added subscribers: cfe-commits, gamesh411, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, szepet, whisperity.
Herald added a project: clang.

The more entries we have in `AnalyzerOptions::ConfigTable`, the more helpful `debug.ConfigDumper` is. With this patch, I'm pretty confident that it'll now emit the entire state of the analyzer, minus the frontend flags.

It would be nice to reserve the config table specifically to checker options only, as storing the regular analyzer configs is kinda redundant.


Repository:
  rC Clang

https://reviews.llvm.org/D57922

Files:
  include/clang/StaticAnalyzer/Core/AnalyzerOptions.h
  lib/StaticAnalyzer/Core/AnalyzerOptions.cpp
  lib/StaticAnalyzer/Frontend/CheckerRegistry.cpp
  test/Analysis/analyzer-config.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D57922.185853.patch
Type: text/x-patch
Size: 8141 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190207/f71822d9/attachment.bin>


More information about the cfe-commits mailing list