[GWP-ASan] Remove thread clamping in tests.
It's better and easier for us to just have threads contend against each
other in the tests if it's more than the maximum supported number of
hardware threads available.
Specifically, the recoverable test fails on Android because the
GTEST_SKIP in a called function, and it only properly works from the
TEST_* harness function. Android tests run on cuttlefish, which can be a
single core with two hyperthreads.
Reviewed By: fmayer
Differential Revision: https://reviews.llvm.org/D143221
GitOrigin-RevId: 10939d1d580b9d3c9c2f3539c6bdb39f408179c0
diff --git a/tests/recoverable.cpp b/tests/recoverable.cpp
index a4c5c3f..adc9731 100644
--- a/tests/recoverable.cpp
+++ b/tests/recoverable.cpp
@@ -182,9 +182,6 @@
std::atomic<bool> StartingGun{false};
std::vector<std::thread> Threads;
constexpr unsigned kNumThreads = 4;
- if (std::thread::hardware_concurrency() < kNumThreads) {
- GTEST_SKIP() << "Not enough threads to run this test";
- }
char *Ptr = static_cast<char *>(AllocateMemory(*GPA));
diff --git a/tests/thread_contention.cpp b/tests/thread_contention.cpp
index 0992b97..26ccd8e 100644
--- a/tests/thread_contention.cpp
+++ b/tests/thread_contention.cpp
@@ -44,12 +44,8 @@
void runThreadContentionTest(unsigned NumThreads, unsigned NumIterations,
gwp_asan::GuardedPoolAllocator *GPA) {
-
std::atomic<bool> StartingGun{false};
std::vector<std::thread> Threads;
- if (std::thread::hardware_concurrency() < NumThreads) {
- NumThreads = std::thread::hardware_concurrency();
- }
for (unsigned i = 0; i < NumThreads; ++i) {
Threads.emplace_back(asyncTask, GPA, &StartingGun, NumIterations);