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

Alexander Kornienko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jan 14 14:28:14 PST 2017


alexfh added a comment.

In https://reviews.llvm.org/D28729#646555, @aaron.ballman wrote:

> 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,


Yes, a comment to that effect near the relevant code wouldn't hurt. Or do you have other suggestions?

>   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?

As discussed with SA folks, alpha checkers are convenient for them to develop new checks, but shouldn't be exposed to users. Some of these experimental checkers might deserve being moved out of alpha or removed completely, but that should be reported to and discussed with the SA maintainers.


https://reviews.llvm.org/D28729





More information about the cfe-commits mailing list