[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