From 3da511f1c71ea480a0c4065f8c0ef04a82930feb Mon Sep 17 00:00:00 2001 From: Harini Date: Mon, 26 Aug 2024 21:57:43 +0000 Subject: [PATCH 1/2] 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 ec4a03c95a635..09a64b043afc1 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 From a199e779aa886491fd15c9363274fe328190b2f6 Mon Sep 17 00:00:00 2001 From: Harini Date: Mon, 26 Aug 2024 23:37:33 +0000 Subject: [PATCH 2/2] Added a comment saying why the file needs REQUIRES: shell. --- compiler-rt/test/fuzzer/focus-function.test | 1 + 1 file changed, 1 insertion(+) diff --git a/compiler-rt/test/fuzzer/focus-function.test b/compiler-rt/test/fuzzer/focus-function.test index 09a64b043afc1..64fd5eebb2389 100644 --- a/compiler-rt/test/fuzzer/focus-function.test +++ b/compiler-rt/test/fuzzer/focus-function.test @@ -1,6 +1,7 @@ # Tests -focus_function # # TODO: don't require linux. +# Requires full shell support for the `for` loop syntax. # REQUIRES: shell, linux UNSUPPORTED: target=aarch64{{.*}}