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

Dávid Bolvanský via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 3 09:02:26 PDT 2018


xbolva00 added a comment.

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

> > I think the premise is a bit off the mark. It's not that these are not for the common user -- it's that they're simply not ready for users at all.
>
> Then why was it merged into clang in the first place? It seems like the whole point of merging it into clang is to get user feedback.
>
> Furthermore, I am trying to update the conversion checker to report more errors, and I would like it to be exposed to users so I can find or fix the FPs.


If users cannot find / use it, you will not get any feedback. How can they be better checkers if no feedback received?

If nobody should use it, remove this "dead" code


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D46159





More information about the cfe-commits mailing list