[PATCH] D54141: [clang-tidy] add deduplication support for run-clang-tidy.py

Tue Nov 6 13:05:18 PST 2018

steveire added a comment.

In https://reviews.llvm.org/D54141#1288930, @JonasToth wrote:

> > Do you understand the proposal now?
>
> Yes better, I was under the impression that `clang-apply-replaments` is run on the end and the YAMLs are kept until then. Now its clear.
>  I assume `--issue-diags` produce the same result as the normal diagnostic engine. That could work, yes.

Great, glad we got that misunderstanding sorted out.

> clang-tidy does not have a `quiet` mode though. It has the `-quiet` option which just does not emit how many warnings were created and suppressed.
>  Do you have these things already in the pipeline?

No, I have not started on these.

At least the clang-tidy quiet mode is trivial to implement. Maybe instead of `--quiet` we could have `--stdout=<output_format>` where `output_format` can be one of `none`, `diag`, `yaml` and in the future possibly `json` (requested here: http://lists.llvm.org/pipermail/cfe-dev/2018-October/059944.html) or `cbor`, to address the binary output suggestion from @hokein.

Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D54141