From 5906b1458601129876864d08274418eb2ee62e9e Mon Sep 17 00:00:00 2001 From: Matthew Parkinson Date: Mon, 9 May 2022 09:46:24 +0100 Subject: [PATCH] Out-of-memory can fail silently If this test fails to allocate memory, that should not cause the test to fail. The 'abort' was added previously to confirm a infrequent failure was caused by out-of-memory causing the test to assign to nullptr. This was confirmed in a CI run, and now the test can be made to ignore allocation failure. --- src/test/perf/contention/contention.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/test/perf/contention/contention.cc b/src/test/perf/contention/contention.cc index 94a45c5..bcea629 100644 --- a/src/test/perf/contention/contention.cc +++ b/src/test/perf/contention/contention.cc @@ -90,7 +90,7 @@ void test_tasks_f(size_t id) else { std::cout << "Failed to allocate " << size << " bytes" << std::endl; - abort(); + // Continue as this is not an important failure. } size_t* out =