[libcxx-commits] [libcxxabi] [libcxxabi][test][VE] Relax test configucation (PR #67387)

Kazushi Marukawa via libcxx-commits libcxx-commits at lists.llvm.org
Mon Sep 25 18:14:42 PDT 2023


https://github.com/kaz7 created https://github.com/llvm/llvm-project/pull/67387

Relax test configuration for VE.  VE limits the number of threads per a process up to 64.  So, it is needed to relax number of threads test configuration.  Otherwise, we get temporary resource exhausted errors at thread creation.

>From e786bb53caa0ac992045a77b9e0e25285c098093 Mon Sep 17 00:00:00 2001
From: "Kazushi (Jam) Marukawa" <marukawa at nec.com>
Date: Sun, 24 Sep 2023 15:53:08 +0200
Subject: [PATCH] [libcxxabi][test][VE] Relax test configucation

Relax test configuration for VE.  VE limits the number of threads
per a process up to 64.  So, it is needed to relax number of threads
test configuration.
---
 libcxxabi/test/guard_threaded_test.pass.cpp | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/libcxxabi/test/guard_threaded_test.pass.cpp b/libcxxabi/test/guard_threaded_test.pass.cpp
index 78dafba7f7ecd9c..6fed14204717dcc 100644
--- a/libcxxabi/test/guard_threaded_test.pass.cpp
+++ b/libcxxabi/test/guard_threaded_test.pass.cpp
@@ -26,6 +26,7 @@
 
 using namespace __cxxabiv1;
 
+#ifndef __ve__
 // Misc test configuration. It's used to tune the flakyness of the test.
 // ThreadsPerTest - The number of threads used
 constexpr int ThreadsPerTest = 10;
@@ -33,6 +34,13 @@ constexpr int ThreadsPerTest = 10;
 constexpr int ConcurrentRunsPerTest = 10;
 // The number of times to rerun each test.
 constexpr int TestSamples = 50;
+#else
+// VE limits the number of threads per a process up to 64.  Above configuration
+// causes exhausted resource error at thread creation.
+constexpr int ThreadsPerTest = 4;
+constexpr int ConcurrentRunsPerTest = 8;
+constexpr int TestSamples = 10;
+#endif
 
 
 



More information about the libcxx-commits mailing list