[PATCH] D57855: [analyzer] Reimplement checker options
Kristóf Umann via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat Mar 16 10:52:45 PDT 2019
Szelethus updated this revision to Diff 190977.
Szelethus edited the summary of this revision.
Szelethus added a comment.
- Moved every non-related change to smaller patches, this should ease **a lot** on reviewers.
- Now processing options once all checkers are added to the registry. This is important, because I use binary searches to find the checkers and packages that need to be modified -- if a plugin however called `CheckerRegistry::add*Option`, it would've cause an assertion failure.
In D57855#1392560 <https://reviews.llvm.org/D57855#1392560>, @xazax.hun wrote:
> We have `examples/analyzer-plugin`. I would prefer to add an example option to the example plugin so people do see how to do this when they are registering a checker from a plugin.
Coming in a separate patch later!
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57855/new/
https://reviews.llvm.org/D57855
Files:
include/clang/Basic/DiagnosticCommonKinds.td
include/clang/StaticAnalyzer/Checkers/CheckerBase.td
include/clang/StaticAnalyzer/Checkers/Checkers.td
include/clang/StaticAnalyzer/Frontend/CheckerRegistry.h
lib/Frontend/CompilerInvocation.cpp
lib/StaticAnalyzer/Frontend/CheckerRegistry.cpp
test/Analysis/disable-all-checks.c
test/Analysis/invalid-checker-option.c
utils/TableGen/ClangSACheckersEmitter.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D57855.190977.patch
Type: text/x-patch
Size: 33853 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190316/60877511/attachment-0001.bin>
More information about the cfe-commits
mailing list