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

via libcxx-commits libcxx-commits at lists.llvm.org
Mon Sep 25 18:15:47 PDT 2023


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-libcxxabi

<details>
<summary>Changes</summary>

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.

---
Full diff: https://github.com/llvm/llvm-project/pull/67387.diff


1 Files Affected:

- (modified) libcxxabi/test/guard_threaded_test.pass.cpp (+8) 


``````````diff
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
 
 
 

``````````

</details>


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


More information about the libcxx-commits mailing list