[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