[llvm] [llvm][test] remove non-posix grep option from many-instructions.s (PR #154997)
David Tenty via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 22 10:48:41 PDT 2025
https://github.com/daltenty created https://github.com/llvm/llvm-project/pull/154997
Not everything uses the built in shell yet, so this non-posix grep option gets passed to the system grep on AIX and causes failures.
The `-o` option just restricts the amount of output in the matching line to just the pattern, so it seems safe to drop here as we will still get the match we are looking for.
>From aad027fb6c03608c6f59aa9af20dc4f563c032e7 Mon Sep 17 00:00:00 2001
From: David Tenty <daltenty at ibm.com>
Date: Fri, 22 Aug 2025 13:36:47 -0400
Subject: [PATCH] [llvm][test] remove non-posix grep option from
many-instructions.s
Not everything uses the built in shell yet, so this non-posix grep option gets passed to the system grep on AIX and causes failures.
The `-o` option just restricts the amount of output in the matching line to just the pattern, so it seems safe to drop here as we will still get the match we are looking for.
---
llvm/test/MC/ELF/many-instructions.s | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/llvm/test/MC/ELF/many-instructions.s b/llvm/test/MC/ELF/many-instructions.s
index 7c13c0d63d536..b5691b5e17121 100644
--- a/llvm/test/MC/ELF/many-instructions.s
+++ b/llvm/test/MC/ELF/many-instructions.s
@@ -1,6 +1,6 @@
## Checks the size of an internal MC structure that is different on 32-bit.
# REQUIRES: asserts, llvm-64-bits
-# RUN: llvm-mc -filetype=obj -triple=x86_64 %s -o /dev/null -debug-only=mc-dump 2>&1 | grep -E -o '[0-9]+ Data Size:[0-9]+' | FileCheck %s
+# RUN: llvm-mc -filetype=obj -triple=x86_64 %s -o /dev/null -debug-only=mc-dump 2>&1 | grep -E '[0-9]+ Data Size:[0-9]+' | FileCheck %s
## Test that encodeInstruction may cause a new fragment to be created.
# CHECK: 0 Data Size:16220
More information about the llvm-commits
mailing list