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

Kazushi Marukawa via libcxx-commits libcxx-commits at lists.llvm.org
Tue Sep 26 01:47:21 PDT 2023


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

>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 1/2] [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
 
 
 

>From 920414af321ac0e5ea483f4bddaab9e6c6fe98d7 Mon Sep 17 00:00:00 2001
From: "Kazushi (Jam) Marukawa" <marukawa at nec.com>
Date: Tue, 26 Sep 2023 17:46:43 +0900
Subject: [PATCH 2/2] Applied clang-format

---
 libcxxabi/test/guard_threaded_test.pass.cpp | 2 --
 1 file changed, 2 deletions(-)

diff --git a/libcxxabi/test/guard_threaded_test.pass.cpp b/libcxxabi/test/guard_threaded_test.pass.cpp
index 6fed14204717dcc..8a5df2e2b88b51d 100644
--- a/libcxxabi/test/guard_threaded_test.pass.cpp
+++ b/libcxxabi/test/guard_threaded_test.pass.cpp
@@ -42,8 +42,6 @@ constexpr int ConcurrentRunsPerTest = 8;
 constexpr int TestSamples = 10;
 #endif
 
-
-
 void BusyWait() {
   std::this_thread::yield();
 }



More information about the libcxx-commits mailing list