[clang-tools-extra] Allow to pass config file to clang-tidy-diff (PR #75457)

Michael Lettrich via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 18 00:12:38 PST 2023


https://github.com/MichaelLettrich updated https://github.com/llvm/llvm-project/pull/75457

>From 382a8a5355b06f191941099c1eac029dbb9d4bb4 Mon Sep 17 00:00:00 2001
From: Michael Lettrich <michael.lettrich at cern.ch>
Date: Thu, 14 Dec 2023 11:31:28 +0100
Subject: [PATCH 1/2] Allow to pass config file to clang-tidy-diff

    Adds a `-config-file` command line option that passes on the path of .`clang-tidy` or custom config file to the `clang-tidy` executable.
---
 clang-tools-extra/clang-tidy/tool/clang-tidy-diff.py | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/clang-tools-extra/clang-tidy/tool/clang-tidy-diff.py b/clang-tools-extra/clang-tidy/tool/clang-tidy-diff.py
index 8817e2914f6e25..d96b3450fdbe81 100755
--- a/clang-tools-extra/clang-tidy/tool/clang-tidy-diff.py
+++ b/clang-tools-extra/clang-tidy/tool/clang-tidy-diff.py
@@ -173,6 +173,12 @@ def main():
         help="checks filter, when not specified, use clang-tidy " "default",
         default="",
     )
+    parser.add_argument(
+        "-config-file",
+        dest="config_file",
+        help="Specify the path of .clang-tidy or custom config file",
+        default="",
+    )
     parser.add_argument("-use-color", action="store_true", help="Use colors in output")
     parser.add_argument(
         "-path", dest="build_path", help="Path used to read a compile command database."
@@ -313,6 +319,8 @@ def main():
         common_clang_tidy_args.append("-fix")
     if args.checks != "":
         common_clang_tidy_args.append("-checks=" + args.checks)
+    if args.config_file != "":
+        common_clang_tidy_args.append("-config-file=" + args.config_file)
     if args.quiet:
         common_clang_tidy_args.append("-quiet")
     if args.build_path is not None:

>From 9ef2829e44f2b8ad27b58e74c4068f2c72aba69b Mon Sep 17 00:00:00 2001
From: Michael Lettrich <MichaelLettrich at users.noreply.github.com>
Date: Mon, 18 Dec 2023 09:12:30 +0100
Subject: [PATCH 2/2] Add to ReleaseNotes.rst

Document changes to clang-tidy-diff in release notes.
---
 clang-tools-extra/docs/ReleaseNotes.rst | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/clang-tools-extra/docs/ReleaseNotes.rst b/clang-tools-extra/docs/ReleaseNotes.rst
index 6d91748e4cef18..7c568d1e94d9f4 100644
--- a/clang-tools-extra/docs/ReleaseNotes.rst
+++ b/clang-tools-extra/docs/ReleaseNotes.rst
@@ -128,6 +128,11 @@ Improvements to clang-tidy
   as a value for `-export-fixes` to export individual yaml files for each
   compilation unit.
 
+- Improved :program:`clang-tidy-diff.py` script. It now accepts a `-config-file` option
+  to pass in a configuration file to :program:`clang-tidy`. This option corresponds
+  to the `--config-file` option in :program:`clang-tidy`.
+
+
 New checks
 ^^^^^^^^^^
 



More information about the cfe-commits mailing list