[compiler-rt] Add REQUIRES: shell to focus-function.test (PR #106150)

via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 26 15:14:58 PDT 2024


https://github.com/Harini0924 created https://github.com/llvm/llvm-project/pull/106150

 This patch adds `REQUIRES: shell` to the `focus-function.test` because the lit internal shell does not support the for loop syntax. This will make the test file unsupported when running llvm-lit with its internal shell implementation, which is enabled by turning on the `LIT_USE_INTERNAL_SHELL=1`. 

fixes: #106111

>From 3da511f1c71ea480a0c4065f8c0ef04a82930feb Mon Sep 17 00:00:00 2001
From: Harini <harinidonthula at google.com>
Date: Mon, 26 Aug 2024 21:57:43 +0000
Subject: [PATCH] Add REQUIRES: shell to focus-function.test

This patch add REQUIRES: shell to focus-function.test because lit internal shell
can't support the for loop syntax in this test.
---
 compiler-rt/test/fuzzer/focus-function.test | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/compiler-rt/test/fuzzer/focus-function.test b/compiler-rt/test/fuzzer/focus-function.test
index ec4a03c95a6355..09a64b043afc11 100644
--- a/compiler-rt/test/fuzzer/focus-function.test
+++ b/compiler-rt/test/fuzzer/focus-function.test
@@ -1,7 +1,7 @@
 # Tests -focus_function
 #
 # TODO: don't require linux.
-# REQUIRES: linux
+# REQUIRES: shell, linux
 UNSUPPORTED: target=aarch64{{.*}}
 
 RUN: %cpp_compiler %S/OnlySomeBytesTest.cpp -o %t-exe



More information about the llvm-commits mailing list