[compiler-rt] ddd2257 - [TSan] Fix incorrect expansion of `%deflake` lit substitution.

Dan Liew via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 19 13:25:57 PST 2020


Author: Dan Liew
Date: 2020-02-19T13:23:26-08:00
New Revision: ddd2257f48a83843aa35e7cbd337c3fe0c103598

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

LOG: [TSan] Fix incorrect expansion of `%deflake` lit substitution.

dadc214e4d9d09a8a7a9f15780c1201c18f93e05 introduced a change to
`%deflake` to support a configurable threshold but the patch forgot
to add a trailing space.

Added: 
    

Modified: 
    compiler-rt/test/tsan/lit.cfg.py

Removed: 
    


################################################################################
diff  --git a/compiler-rt/test/tsan/lit.cfg.py b/compiler-rt/test/tsan/lit.cfg.py
index a07adaa0d657..9ef1068fb35a 100644
--- a/compiler-rt/test/tsan/lit.cfg.py
+++ b/compiler-rt/test/tsan/lit.cfg.py
@@ -75,7 +75,7 @@ def build_invocation(compile_flags):
 # Define CHECK-%os to check for OS-dependent output.
 config.substitutions.append( ('CHECK-%os', ("CHECK-" + config.host_os)))
 
-config.substitutions.append( ("%deflake ", os.path.join(os.path.dirname(__file__), "deflake.bash") + " " + config.deflake_threshold))
+config.substitutions.append( ("%deflake ", os.path.join(os.path.dirname(__file__), "deflake.bash") + " " + config.deflake_threshold + " "))
 
 # Default test suffixes.
 config.suffixes = ['.c', '.cpp', '.m', '.mm']


        


More information about the llvm-commits mailing list