[PATCH] D65065: [clang-tidy] Possibility of displaying duplicate warnings

Tibor Brunner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 22 01:08:03 PDT 2019


bruntib created this revision.
bruntib added reviewers: alexfh, xazax.hun, Szelethus.
Herald added subscribers: cfe-commits, mgrang, rnkovacs, whisperity.
Herald added a project: clang.

In case a checker is registered multiple times as an alias, the emitted warnings are uniqued by the report message. However, it is random which checker name is included in the warning. When processing the output of clang-tidy this behavior caused some problems. So in this commit clang-tidy has been extended with a --duplicate-reports flag which prevents uniquing the warnings and both checker's messages are printed under their name.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D65065

Files:
  clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.cpp
  clang-tools-extra/clang-tidy/ClangTidyOptions.cpp
  clang-tools-extra/clang-tidy/ClangTidyOptions.h
  clang-tools-extra/clang-tidy/tool/ClangTidyMain.cpp
  clang-tools-extra/test/clang-tidy/duplicate-reports-with.cpp
  clang-tools-extra/test/clang-tidy/duplicate-reports-without.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D65065.211023.patch
Type: text/x-patch
Size: 5722 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190722/da8cc358/attachment.bin>


More information about the cfe-commits mailing list