[PATCH] D66042: [analyzer] Analysis: "Disable" core checkers
Artem Dergachev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Aug 9 20:06:27 PDT 2019
NoQ added inline comments.
================
Comment at: clang/include/clang/StaticAnalyzer/Core/AnalyzerOptions.h:192-197
+ /// Pair of checker name and enable/disable to do analysis.
+ std::vector<std::pair<std::string, bool>> CheckerAnalysisVector;
+
+ /// Vector of checker names to do not emit warnings.
+ std::vector<std::string> CheckerSilenceVector;
----------------
`CheckerAnalysisVector` sounds like a vector of checkers that will be subject to analysis. But in reality they are the ones that are analyzing and nobody is analyzing them.
The old name isn't very expressive either, but at least it sounds cool >.<
Maybe `EnabledCheckers`, `SilencedCheckers`?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D66042/new/
https://reviews.llvm.org/D66042
More information about the cfe-commits
mailing list