NoQ accepted this revision.
NoQ added a comment.

> Checker silencing used to kick in after all of these. Now it does before any 
> of them :^)

This is probably the main benefit of the patch. Running visitors is cheap most 
of the time but it can be expensive.

Silencing path-insensitive checkers isn't particularly useful because they 
don't interact with other checkers so it's entirely equivalent to disabling 
them. Unless, well, somebody someday introduces an interaction. Still, it's 
great to have the interface work consistently.


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

https://reviews.llvm.org/D102914

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D102914: [analyzer... Artem Dergachev via Phabricator via cfe-commits

Reply via email to