[compiler-rt] [compiler-rt][tests] Removed the use of parentheses in compiler-rt tests with lit internal shell (PR #105729)
via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 26 08:24:27 PDT 2024
================
@@ -23,7 +23,9 @@
// RUN: %env_ubsan_opts=halt_on_error=1 not %run %t nN 2>&1 | FileCheck %s --check-prefix=CHECK-NULL-MEMFUN --strict-whitespace
// RUN: %env_ubsan_opts=print_stacktrace=1 %run %t dT 2>&1 | FileCheck %s --check-prefix=CHECK-DYNAMIC --allow-unused-prefixes --check-prefix=CHECK-%os-DYNAMIC --strict-whitespace
-// RUN: (echo "vptr_check:S"; echo "vptr_check:T"; echo "vptr_check:U") > %t.supp
+// RUN: echo "vptr_check:S" > %t.supp
+// RUN: echo "vptr_check:T" >> %t.supp
+// RUN: echo "vptr_check:U" >> %t.supp
----------------
Harini0924 wrote:
I used `echo -e` in a single `RUN` line to create the suppression file, which simplifies the process and avoids the need for multiple subshell invocations. This keeps the script efficient and easy to maintain. Additionally, this approach is better because it minimizes changes to the file, unlike when using `split-file`. What are your thoughts on this approach?
https://github.com/llvm/llvm-project/pull/105729
More information about the llvm-commits
mailing list