[compiler-rt] [compiler-rt][test] Rewrote test to remove curly braces (PR #105696)

Connie Zhu via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 27 18:05:01 PDT 2024


================
@@ -1,7 +1,10 @@
 // RUN: %clangxx_asan -xc++ -shared -fPIC -o %t.so - < %s
 // RUN: %clang_asan %s -o %t.out -ldl
 //
-// RUN: { env ASAN_OPTIONS=verbosity=1 %t.out %t.so || : ; } 2>&1 | FileCheck %s
+// The program can potentially return a non-zero exit code, so use || : to
+// ensure command returns true
+// RUN: env ASAN_OPTIONS=verbosity=1 %t.out %t.so &> %t_env || :
----------------
connieyzhu wrote:

I replaced `||:` with `not` in my most recent commit. The tests are passing when I run things locally - still need to wait for builders though.

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


More information about the llvm-commits mailing list