[PATCH] D59449: [clang-tidy] Integrate clang-tidy-diff.py machinery into run-clang-tidy.py
Zinovy Nis via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Apr 1 05:11:55 PDT 2019
zinovy.nis added a comment.
> Why not just use clang-tidy-diff.py? The clang-tidy-diff.py script has a distinct and somewhat self-documenting name and a very specific purpose (find clang-tidy regressions in a patch), while run-clang-tidy.py is more focused on running over larger bodies of code with a purpose of analyzing or cleaning up. Is there any benefit in having all functionality in run-clang-tidy.py?
Both scripts have almost the same implementation, almost the same syntax (at least after -j and -timeout we introduced). So why not merge them.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59449/new/
https://reviews.llvm.org/D59449
More information about the cfe-commits
mailing list