[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
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 33853 bytes
Desc: not available
More information about the cfe-commits