[compiler-rt] cac25e4 - Adapt `tsan/flush_memory.cpp` to run on non-local platforms.

Dan Liew via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 29 10:39:44 PDT 2021


Author: Dan Liew
Date: 2021-09-29T10:39:38-07:00
New Revision: cac25e40428de19bd78b821dce47c36ebca2ce6c

URL: https://github.com/llvm/llvm-project/commit/cac25e40428de19bd78b821dce47c36ebca2ce6c
DIFF: https://github.com/llvm/llvm-project/commit/cac25e40428de19bd78b821dce47c36ebca2ce6c.diff

LOG: Adapt `tsan/flush_memory.cpp` to run on non-local platforms.

ad890aa2327feb6b6aee676fe85b2352fba2403e landed a test without
using the `%run` prefix which means the test fails to run for
platforms that need it (e.g. iOS simulators).

This patch adds the `%run` prefix. While we're here also split
the single `RUN` line into two to make debugging easier.

rdar://83637296

Differential Revision: https://reviews.llvm.org/D110734

Added: 
    

Modified: 
    compiler-rt/test/tsan/flush_memory.cpp

Removed: 
    


################################################################################
diff  --git a/compiler-rt/test/tsan/flush_memory.cpp b/compiler-rt/test/tsan/flush_memory.cpp
index a68ce0ab3b840..4ab48fef21915 100644
--- a/compiler-rt/test/tsan/flush_memory.cpp
+++ b/compiler-rt/test/tsan/flush_memory.cpp
@@ -1,4 +1,5 @@
-// RUN: %clangxx_tsan -O1 %s -o %t && %env_tsan_opts="flush_memory_ms=1 flush_symbolizer_ms=1 memory_limit_mb=1" %deflake %t | FileCheck %s
+// RUN: %clangxx_tsan -O1 %s -o %t
+// RUN: %env_tsan_opts="flush_memory_ms=1 flush_symbolizer_ms=1 memory_limit_mb=1" %deflake %run %t | FileCheck %s
 #include "test.h"
 
 long X, Y;


        


More information about the llvm-commits mailing list