From: Dan Liew Date: Fri, 1 Oct 2021 20:04:13 +0000 (-0700) Subject: Use standard separator for TSan options in `stress.cpp` test case. X-Git-Tag: upstream/15.0.7~29811 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d6a4294d1362b37f07a96c8ca6d18e260d44bcd6;p=platform%2Fupstream%2Fllvm.git Use standard separator for TSan options in `stress.cpp` test case. Use of space as a separator for options is problematic for wrapper scripts (i.e. implementations of `%run`) that have to marshall environment variables to target different than the host. Rather than requiring every implementation of `%run` to support spaces in `TSAN_OPTIONS` it is simpler to fix this single test case. rdar://83637067 Differential Revision: https://reviews.llvm.org/D110967 --- diff --git a/compiler-rt/test/tsan/stress.cpp b/compiler-rt/test/tsan/stress.cpp index d678395..3656223 100644 --- a/compiler-rt/test/tsan/stress.cpp +++ b/compiler-rt/test/tsan/stress.cpp @@ -1,4 +1,4 @@ -// RUN: %clangxx_tsan -O1 %s -o %t && %env_tsan_opts="flush_memory_ms=1 flush_symbolizer_ms=1 memory_limit_mb=1" %run %t 2>&1 | FileCheck %s +// RUN: %clangxx_tsan -O1 %s -o %t && %env_tsan_opts=flush_memory_ms=1:flush_symbolizer_ms=1:memory_limit_mb=1 %run %t 2>&1 | FileCheck %s #include "test.h" #include #include