Szelethus added a comment.

In D60925#1489034 <https://reviews.llvm.org/D60925#1489034>, @o.gyorgy wrote:

> Great, at least the users will not enable the debug checkers by accident!
>  We will check on the CodeChecker side if any configuration needs to be 
> updated.
>
> LGTM


I committed after checking whether this works with `CodeChecker`. I listed the 
enabled-by-default list with `CodeChecker checkers --only-enabled`, and made 
sure that the list didn't change after this patch. I also made sure that 
`CodeChecker checkers --only-enabled` is consistent with `clang [blahblah] 
-analyzer-list-enabled-checkers`, meaning that what `CodeChecker` requested was 
actually enabled, and nothing else (except modeling checkers, of course).

I guess adding a strictly developer only option to list the hidden checkers as 
well might be desirable, if we need to disable certain modeling checkers to get 
rid of some crashes.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60925



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

Reply via email to