[PATCH] D46159: [clang-tidy] Add a flag to enable alpha checkers

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 3 08:06:54 PDT 2018


lebedev.ri added a comment.

In https://reviews.llvm.org/D46159#1086463, @pfultz2 wrote:

> > As Devin says (and as we discussed this with Anna Zaks) alpha checkers are still in development, so we don't want to expose them to the users, even very curious ones.
>
> Then why do we make them available with `clang --analyze`? If the plan is not to expose them to the users at all, they should be removed from the codebase, as they are just sitting there bit-rotting.
>
> Ideally, I see no problem exposing them to users. This will allow more users to run them on their codebases and submit issues or patches for the problems they find.


Just so i'm perfectly clear, i find this "they are not for common user, so let's not expose them at all" approach very Gnome-like, and not really appropriate for LLVM.
With that approach, there should also not be a https://clang.llvm.org/docs/ClangCommandLineReference.html#cmdoption-clang-fexperimental-new-pass-manager and maybe some other flags.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D46159





More information about the cfe-commits mailing list