[PATCH] D54438: [analyzer][WIP] Reimplement dependencies between checkers

Umann Kristóf via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 15 09:45:28 PST 2018


Szelethus added a comment.
Herald added a subscriber: gamesh411.

In https://reviews.llvm.org/D54438#1297602, @NoQ wrote:

> Hmm, makes sense. Maybe `static bool BlahBlahChecker::isApplicable(const LangOpts &LO)` or something like that.
>
> Btw, maybe instead of default constructor and `->setup(Args)` method we could stuff all of the `setup(Args)`'s arguments into the one-and-only constructor for the checker?


Neat. Make all checkers have only a single constructor, expecting a `const CheckerManager &`! Since `getCurrentName` and the like will be thing of the past, for once, maybe we could also initialize checker options in constructors too. Hopefully.


Repository:
  rC Clang

https://reviews.llvm.org/D54438





More information about the cfe-commits mailing list