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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to