[clang-tools-extra] [clang-tidy] Use --match-full-lines instead of --strict-whitespace in check_clang_tidy (PR #133756)

Nicolas van Kempen via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 31 12:14:17 PDT 2025


================
@@ -245,15 +246,15 @@ def check_no_diagnosis(self, clang_tidy_output: str) -> None:
 
     def check_fixes(self) -> None:
         if self.has_check_fixes:
-            try_run(
-                [
-                    "FileCheck",
-                    "-input-file=" + self.temp_file_name,
-                    self.input_file_name,
-                    "-check-prefixes=" + ",".join(self.fixes.prefixes),
-                    "-strict-whitespace",
-                ]
-            )
+            command = [
+                "FileCheck",
+                "--input-file=" + self.temp_file_name,
+                self.input_file_name,
+                "--check-prefixes=" + ",".join(self.fixes.prefixes),
+            ]
+            if not self.match_partial_fixes:
----------------
nicovank wrote:

Right, that makes more sense, especially since (at least one) tests work when enabled but fail when disabled. I initially thought it was strictly less aggressive to not enable it anyway but maybe not.

https://github.com/llvm/llvm-project/pull/133756


More information about the cfe-commits mailing list