[compiler-rt] [compiler-rt][test] Added REQUIRES:shell to fuzzer test with for-loop (PR #105557)

via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 21 10:33:27 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-compiler-rt-sanitizer

Author: Connie Zhu (connieyzhu)

<details>
<summary>Changes</summary>

This patch makes the features_dir.test file require a shell when running. 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 environment variable. Lit's internal shell currently does not support for-loop syntax.

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


1 Files Affected:

- (modified) compiler-rt/test/fuzzer/features_dir.test (+1-1) 


``````````diff
diff --git a/compiler-rt/test/fuzzer/features_dir.test b/compiler-rt/test/fuzzer/features_dir.test
index c6beec01bc3ab2..9298fdace6cd7c 100644
--- a/compiler-rt/test/fuzzer/features_dir.test
+++ b/compiler-rt/test/fuzzer/features_dir.test
@@ -1,5 +1,5 @@
 # Tests -features_dir=F
-# REQUIRES: linux
+# REQUIRES: shell
 RUN: %cpp_compiler %S/SimpleTest.cpp -o %t-SimpleTest
 RUN: rm -rf %t-C %t-F
 RUN: mkdir %t-C %t-F

``````````

</details>


https://github.com/llvm/llvm-project/pull/105557


More information about the llvm-commits mailing list