NoQ added a comment.

Thx!~



================
Comment at: lib/StaticAnalyzer/Frontend/CheckerRegistry.cpp:527-530
+    // developer checkers even in the alpha output. For example,
+    // alpha.cplusplus.IteratorModeling is a modeling checker, hence it's 
hidden
+    // by default, and users (even when the user is a developer of an alpha
+    // checker) shouldn't normally tinker with whether they should be enabled.
----------------
Szelethus wrote:
> Szelethus wrote:
> > NoQ wrote:
> > > Hmm, just thought about this: by this logic, should we hide all `core` 
> > > checkers? Because we don't really support users tinkering with whether 
> > > they should be enabled :)
> > Thats actually a very cool idea. We should totally do that!
> Well umm. In a sense, sure, but wouldn't we want to display the checker 
> descriptions at least? Let's not rush this for now.
Yup. I guess we just separate models from checkers eventually.


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

https://reviews.llvm.org/D62093



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

Reply via email to