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)


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)
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D145477... Tiwari Abhinav Ashok Kumar via Phabricator via cfe-commits
    • [PATCH] D1... Carlos Galvez via Phabricator via cfe-commits
    • [PATCH] D1... Tiwari Abhinav Ashok Kumar via Phabricator via cfe-commits

Reply via email to