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

Umann Kristóf via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 12 14:36:09 PST 2018


Szelethus added a comment.

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

> Can we reduce this patch to simply introducing the dependency item in `Checkers.td` and using it in, like, one place (eg., `MallocChecker`/`CStringChecker` inter-op) and discuss the rest of the stuff later before we jump into it?


Short answer, no. The original intent was to somehow make sense out of checker options, and look where that got me :D There are no more shortcuts to make, sadly. I strongly believe it'd be better to figure out how those checkers should operate before moving forward with this patch, because I know I wouldn't like to touch any of those checkers once this patch goes through, so let's tough it out first.

Also, doing some checker related work might be refreshing after the never ending refactoring.

I'll brush the dust off some notebooks and see what I can come up with.


Repository:
  rC Clang

https://reviews.llvm.org/D54438





More information about the cfe-commits mailing list