[PATCH] Post-filter clang-tidy diagnostic messages.

Manuel Klimek klimek at google.com
Wed Mar 19 12:54:07 PDT 2014


On Wed, Mar 19, 2014 at 5:35 PM, Alexander Kornienko <alexfh at google.com>wrote:

> Hi klimek,
>
> This patch implements filtering of clang-tidy diagnostic messages by
> the check name, so that "clang-tidy -checks=^llvm-" won't output any clang
> warnings, for example. This is also helpful to run specific static-analyzer
> checks: static analyzer always needs core checks to be enabled, but the
> user may
> be interested only in the checks he asked for.
>
> This patch also exposes warning option names for built-in diagnostics. We
> need
> to have a namespace for these names to avoid collisions and to allow
> convenient
> filtering, so I prefix them with "-W". I'm not sure it's the best thing to
> do,
> and maybe "W" or "clang-diagnostic-" or something like this would be
> better.
>

+1 for clang-diagnostic...


>
> http://llvm-reviews.chandlerc.com/D3121
>
> Files:
>   clang-tidy/ClangTidy.cpp
>   clang-tidy/ClangTidy.h
>   clang-tidy/ClangTidyDiagnosticConsumer.cpp
>   clang-tidy/ClangTidyDiagnosticConsumer.h
>   clang-tidy/ClangTidyModule.cpp
>   test/clang-tidy/diagnostic.cpp
>   unittests/clang-tidy/ClangTidyTest.h
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140319/2d0647f7/attachment.html>


More information about the cfe-commits mailing list