[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