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

Alexander Kornienko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 16 08:56:33 PST 2017

alexfh added a comment.

In https://reviews.llvm.org/D28729#647250, @Prazek wrote:

> Does solution like this works for you? We don't officially support alpha checkers, but it is much easier to check if something is already implemented in static analyzer easily

Is it the only problem you're trying to solve by this patch? If so, it might be better to add an alias/script to run `clang --analyze -Xclang -analyzer-checker=alpha` or even use gcc.godbolt.org (e.g. https://godbolt.org/g/msQigz).

My concern is that if there's a user-visible option (undocumented, unlisted, named `--no-no-no-please-dont-use-this-option` or otherwise discouraged), people will use it (sometimes unintentionally, because, well, they changed their config once to give it a try and then forgot about it for a year), rely on it, complain about the tool, etc. If clang-tidy is used in some sort of a server-side manner (e.g. integrated with code review), misbehaving checkers may cause crashes, and this is problematic, when maintainers have no intention to fix the bugs and well, they explicitly asked not to expose experimental checkers (see http://llvm.org/PR26855, for example).


More information about the cfe-commits mailing list