[llvm] Revert changes to llvm-string stdin test and XFAIL on AIX (PR #160821)
via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 26 00:24:54 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-llvm-binary-utilities
Author: Mark Danial (madanial0)
<details>
<summary>Changes</summary>
Xfail testcase on AIX until echo -n bug is fixed, revert changes made to testcase
---
Full diff: https://github.com/llvm/llvm-project/pull/160821.diff
1 Files Affected:
- (modified) llvm/test/tools/llvm-strings/stdin.test (+5-6)
``````````diff
diff --git a/llvm/test/tools/llvm-strings/stdin.test b/llvm/test/tools/llvm-strings/stdin.test
index 3c60123ca8921..ef6cd131f5879 100644
--- a/llvm/test/tools/llvm-strings/stdin.test
+++ b/llvm/test/tools/llvm-strings/stdin.test
@@ -1,13 +1,13 @@
+; XFAIL: target={{.*}}-aix{{.*}}
+
## Show that llvm-strings can handle stdin input properly.
## Case 1: output with single string.
-RUN: echo -n "abcdefg" > %t
-RUN: llvm-strings %t - | FileCheck %s --check-prefix=CASE1 --implicit-check-not={{.}}
+RUN: echo -n "abcdefg" | llvm-strings - | FileCheck %s --check-prefix=CASE1 --implicit-check-not={{.}}
CASE1: abcdefg
## Case 2: output too short for string.
-RUN: echo -n "abc" > %t
-RUN: llvm-strings %t - | FileCheck %s --implicit-check-not={{.}} --allow-empty
+RUN: echo -n "abc" | llvm-strings - | FileCheck %s --implicit-check-not={{.}} --allow-empty
## Case 3: output with new line.
RUN: printf "abcd\nefgh" | llvm-strings - | FileCheck %s --check-prefix=CASE3 --implicit-check-not={{.}}
@@ -21,5 +21,4 @@ CASE4-NEXT: ghij
CASE4-NEXT: klmn
## Case 5: no file name specified is equivalent to explicitly requesting stdin.
-RUN: echo -n "abcdefg" > %t
-RUN: llvm-strings %t - | FileCheck %s --check-prefix=CASE1 --implicit-check-not={{.}}
+RUN: echo -n "abcdefg" | llvm-strings | FileCheck %s --check-prefix=CASE1 --implicit-check-not={{.}}
``````````
</details>
https://github.com/llvm/llvm-project/pull/160821
More information about the llvm-commits
mailing list