[compiler-rt] [compiler-rt][tests] Removed the use of parentheses in compiler-rt tests with lit internal shell (PR #105729)

Paul Kirth via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 22 14:02:49 PDT 2024


================
@@ -17,7 +17,8 @@ RUN: echo ....E. > %tmp/T2/5
 RUN: echo .....R > %tmp/T2/6
 
 # Check that we can report an error if file size exceeded
-RUN: (ulimit -f 1; not %run %t-FullCoverageSetTest -merge=1 %tmp/T1 %tmp/T2 2>&1 | FileCheck %s --check-prefix=SIGXFSZ)
+RUN: ulimit -f 1
+RUN: not %run %t-FullCoverageSetTest -merge=1 %tmp/T1 %tmp/T2 2>&1 | FileCheck %s --check-prefix=SIGXFSZ
----------------
ilovepi wrote:

`ulimit` here used to execute in the subshell, but now it will affect other commands. Since this use of `ulimit` is going to either require shell or an extension to llvm-lit, we can probably just ignore the parentheses in this file for now.

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


More information about the llvm-commits mailing list