[PATCH] D32294: [clang-tidy] run-clang-tidy.py: check if clang-apply-replacements succeeds

Piotr Padlewski via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 20 17:15:07 PDT 2017


Prazek added a comment.

In https://reviews.llvm.org/D32294#732861, @kuhar wrote:

> After thinking about Piotr's comment, I think that a better way to perform the check would be te invoking clang-apply-replacements with `--version` and seeing if it fails even before running clang-tidy.
>  This way it is possible to save a lot of time when run-clang-tidy.py is run on a big project and apply_fixes fails at the end.
>
> Let me know what you think about this approach.


Sounds good to me!


https://reviews.llvm.org/D32294





More information about the cfe-commits mailing list