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

Alexander Kornienko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 20 17:20:49 PDT 2017


alexfh added a comment.

In https://reviews.llvm.org/D32294#732986, @Prazek wrote:

> 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!


+1


https://reviews.llvm.org/D32294





More information about the cfe-commits mailing list