[PATCH] D69746: [analyzer] FixItHint: Apply and test hints with the Clang Tidy's script

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 22 12:25:37 PST 2019


NoQ added a comment.

In D69746#1756448 <https://reviews.llvm.org/D69746#1756448>, @gribozavr2 wrote:

> > This patch introduces a way to apply the fix-its by the Analyzer:
>
> I'm not sure this option is very useful... I don't know of anyone who uses the same option in Clang or ClangTidy. The only way I know people apply fixits is with the help of IDEs. I am also skeptical that people want to apply *all* fixits. Usually people want to pick a few specific ones, or all fixits of a certain kind; but not everything.
>
> What workflow are you thinking of for this option?


For now this is definitely for testing purposes only. This patch doesn't expose the option in any of the interfaces that are intended for actual users to use (`-analyzer-config` is not one of them).

I don't have any immediate plans on exposing this option to users. That said, the user can always apply fixits of a specific checker by only running that checker (or by only enabling fixits of this checker).


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69746/new/

https://reviews.llvm.org/D69746





More information about the cfe-commits mailing list