llvmbot wrote:

<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang-format

Author: Mark Danial (madanial0)

<details>
<summary>Changes</summary>

https://github.com/llvm/llvm-project/pull/119666 adds the `-strip-trailing-cr` 
flag to diff which is not supported on AIX switch to use the python 
implementation of diff instead

---
Full diff: https://github.com/llvm/llvm-project/pull/120276.diff


1 Files Affected:

- (modified) clang/test/Format/lit.local.cfg (+8) 


``````````diff
diff --git a/clang/test/Format/lit.local.cfg b/clang/test/Format/lit.local.cfg
index 8acf02725d701b..b060c79226cbda 100644
--- a/clang/test/Format/lit.local.cfg
+++ b/clang/test/Format/lit.local.cfg
@@ -1,3 +1,6 @@
+import platform
+import lit.formats
+
 # Suffixes supported by clang-format.
 config.suffixes = [
     ".c",
@@ -19,3 +22,8 @@ config.suffixes = [
     ".td",
     ".test"
 ]
+
+# AIX 'diff' command doesn't support --strip-trailing-cr, but the internal
+# python implementation does, so use that for cross platform compatibility
+if platform.system() == "AIX":
+    config.test_format = lit.formats.ShTest()

``````````

</details>


https://github.com/llvm/llvm-project/pull/120276
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to