[PATCH] D145477: run-clang-tidy.py should only search for the clang-apply-replacements if really needed

2023-03-07 Thread Tiwari Abhinav Ashok Kumar via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG144e2364410c: run-clang-tidy.py should only search for the clang-apply-replacements if really… (authored by Anonymous, committed by aabhinavg). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D145477: run-clang-tidy.py should only search for the clang-apply-replacements if really needed

2023-03-07 Thread Carlos Galvez via Phabricator via cfe-commits
carlosgalvezp accepted this revision. carlosgalvezp added a comment. This revision is now accepted and ready to land. LGTM, `clang_apply_replacements_binary` is only used in a `if args.fix` block. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D145477: run-clang-tidy.py should only search for the clang-apply-replacements if really needed

2023-03-07 Thread Tiwari Abhinav Ashok Kumar via Phabricator via cfe-commits
aabhinavg created this revision. aabhinavg added a reviewer: keith. Herald added subscribers: PiotrZSL, carlosgalvezp. Herald added a reviewer: njames93. Herald added a project: All. aabhinavg requested review of this revision. Herald added a project: clang-tools-extra. Herald added a subscriber: