[compiler-rt] [compiler-rt][tests] Removed the use of parentheses in compiler-rt tests with lit internal shell (PR #105729)
Florian Mayer via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 26 10:13:15 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
----------------
fmayer wrote:
LGTM, but I will note that POSIX actually says `-e` is implementation dependent and that the default should be to interpret the escape characters: https://pubs.opengroup.org/onlinepubs/9799919799/. It appears that `/bin/echo` or bash's builtin do not follow this though :(
https://github.com/llvm/llvm-project/pull/105729
More information about the llvm-commits
mailing list