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

Tiwari Abhinav Ashok Kumar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 7 00:02:44 PST 2023


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: cfe-commits.

run-clang-tidy.py should only search for the clang-apply-replacements if really needed.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D145477

Files:
  clang-tools-extra/clang-tidy/tool/run-clang-tidy.py


Index: clang-tools-extra/clang-tidy/tool/run-clang-tidy.py
===================================================================
--- clang-tools-extra/clang-tidy/tool/run-clang-tidy.py
+++ clang-tools-extra/clang-tidy/tool/run-clang-tidy.py
@@ -300,7 +300,7 @@
                                   build_path)
 
   tmpdir = None
-  if args.fix or (yaml and args.export_fixes):
+  if args.fix:
     clang_apply_replacements_binary = find_binary(
       args.clang_apply_replacements_binary, "clang-apply-replacements",
       build_path)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D145477.502939.patch
Type: text/x-patch
Size: 533 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230307/f1cbc907/attachment.bin>


More information about the cfe-commits mailing list