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

Umann Kristóf via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Nov 24 08:43:54 PST 2018


Szelethus added a comment.

Hmmm, I was wrong a little bit: I realized that if I tinker with with `initializeManager` and `getEnabledCheckers` just a bit more, register checkers straight away, don't need to collect them first, and this would make sure that dependencies are registered before the checkers that depend on them. That means I can get away this patch without even touching the MallocChecker family. `CheckerManager` would receive the `getChecker` function that would assert if the checker isn't already registered, so we could be extra sure this system works.

Buuuut since I spent a lot of time with MallocChecker, I might tickle the splitting problem later on as a weekend project or something.


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54438/new/

https://reviews.llvm.org/D54438





More information about the cfe-commits mailing list