Re: [PATCH] D23060: [analyzer] Show enabled checker list

2016-08-08 Thread Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL278006: [analyzer] Command line option to show enabled checker list. (authored by xazax). Changed prior to commit: https://reviews.llvm.org/D23060?vs=67129&id=67157#toc Repository: rL LLVM https://r

Re: [PATCH] D23060: [analyzer] Show enabled checker list

2016-08-08 Thread Gábor Horváth via cfe-commits
xazax.hun updated this revision to Diff 67129. xazax.hun marked an inline comment as done. xazax.hun added a comment. - Changes according to the review comments. https://reviews.llvm.org/D23060 Files: include/clang/Driver/CC1Options.td include/clang/StaticAnalyzer/Core/AnalyzerOptions.h i

Re: [PATCH] D23060: [analyzer] Show enabled checker list

2016-08-03 Thread Gábor Horváth via cfe-commits
xazax.hun added a comment. In https://reviews.llvm.org/D23060#504832, @NoQ wrote: > Like! > > Do you think that if we rename the ~~bike shed~~ option to > `-analyzer-list-enabled-checkers` it'd be more natural and easy to remember? > (i barely remember how to spell `-analyzer-checker-help`, it'

Re: [PATCH] D23060: [analyzer] Show enabled checker list

2016-08-03 Thread Artem Dergachev via cfe-commits
NoQ added a comment. Like! Do you think that if we rename the ~~bike shed~~ option to `-analyzer-list-enabled-checkers` it'd be more natural and easy to remember? (i barely remember how to spell `-analyzer-checker-help`, it'd be much easier if it was called `-analyzer-list-checkers`). ==

[PATCH] D23060: [analyzer] Show enabled checker list

2016-08-02 Thread Gábor Horváth via cfe-commits
xazax.hun created this revision. xazax.hun added reviewers: zaks.anna, dcoughlin, NoQ. xazax.hun added a subscriber: cfe-commits. This patch adds a command line option to list the checkers that were enabled by analyzer-checker and not disabled by -analyzer-disable-checker. It can be very useful