[compiler-rt] r364366 - Add USan+ASan and UBSan+TSan tests to shadow-memory lit parallelism group.

Dan Liew via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 25 17:35:51 PDT 2019


Author: delcypher
Date: Tue Jun 25 17:35:51 2019
New Revision: 364366

URL: http://llvm.org/viewvc/llvm-project?rev=364366&view=rev
Log:
Add USan+ASan and UBSan+TSan tests to shadow-memory lit parallelism group.

Summary:
Previously we were running these tests without the "shadow-memory"
lit parallelism group even though we run the ASan and TSan tests in
this group to avoid problems with many processes using shadow memory
in parallel.

On my local machine the UBSan+TSan tests would previously timeout
if I set a 30 second per test limit. With this change I no longer
see individual test timeouts.

This change was made in response to the greendragon build bot reporting
individual test timeouts for these tests. Given that the UBSan+ASan and
UBSan+TSan tests did not have a parallelism group previously it's likely
that some other change has caused the performance degradation. However
I haven't been able to track down the cause so until we do, this change
seems reasonable and is in line with what we already do with ASan and
TSan tests.

rdar://problem/51754620

Reviewers: yln, kubamracek, vsk, samsonov

Subscribers: #sanitizers, llvm-commits

Tags: #sanitizers, #llvm

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

Modified:
    compiler-rt/trunk/test/ubsan/lit.common.cfg

Modified: compiler-rt/trunk/test/ubsan/lit.common.cfg
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/ubsan/lit.common.cfg?rev=364366&r1=364365&r2=364366&view=diff
==============================================================================
--- compiler-rt/trunk/test/ubsan/lit.common.cfg (original)
+++ compiler-rt/trunk/test/ubsan/lit.common.cfg Tue Jun 25 17:35:51 2019
@@ -74,3 +74,7 @@ if config.host_os not in ['Linux', 'Darw
 config.available_features.add('arch=' + config.target_arch)
 
 config.excludes = ['Inputs']
+
+if "ubsan-asan" in config.available_features or "ubsan-tsan" in config.available_features:
+  if not config.parallelism_group:
+    config.parallelism_group = 'shadow-memory'




More information about the llvm-commits mailing list