[PATCH] D28729: [clang-tidy] Add -enable-alpha-checks command

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jan 14 14:17:04 PST 2017


aaron.ballman added a comment.

In https://reviews.llvm.org/D28729#646548, @alexfh wrote:

> As discussed with the Static Analyzer maintainers, alpha checkers are completely unsupported and are suitable for very early testing only. We had problems with them routinely, that's why I disabled alpha checkers in clang-tidy completely. I don't think there should be a user-visible way to enable them. Developers can locally change the code to get access to alpha checkers, but released binaries shouldn't provide this possibility.


That's good to know -- should it be documented a bit more explicitly, or perhaps the alpha checks should be removed until they're fit for public consumption? Some of those alpha checks have been in the product for a long time, and if they're so unstable that we cannot expose them in a user-friendly fashion, perhaps they don't belong yet?


https://reviews.llvm.org/D28729





More information about the cfe-commits mailing list