Merge pull request #2271 from DARMA-tasking/2270-fix-asan-error-when-…

…running-perf-tests #2270: Fix ASAN error when running perf tests
DARMA-tasking · Apr 17, 2024 · 80a4ea0 · 80a4ea0
2 parents 084c6c3 + 2afdc00
commit 80a4ea0
Showing 1 changed file with 27 additions and 21 deletions.
diff --git a/tests/perf/common/test_harness_macros.h b/tests/perf/common/test_harness_macros.h
@@ -47,6 +47,8 @@
 #include "test_harness_base.h"
 
 #include <vector>
+#include <memory>
+
 namespace vt { namespace tests { namespace perf { namespace common {
 
 /**
@@ -70,44 +72,48 @@ namespace vt { namespace tests { namespace perf { namespace common {
 
 
 struct PerfTestRegistry{
-  static void AddTest(TestHarnessBase* test){
-    tests_.push_back(test);
+  static void AddTest(std::unique_ptr<TestHarnessBase>&& test) {
+    tests_.push_back(std::move(test));
   }
 
-  static const std::vector<TestHarnessBase*>&
-  GetTests(){
+  static const std::vector<std::unique_ptr<TestHarnessBase>>&
+  GetTests() {
     return tests_;
   }
 
   private:
-  inline static std::vector<TestHarnessBase*> tests_ = {};
+  inline static std::vector<std::unique_ptr<TestHarnessBase>> tests_ = {};
 };
 
-#define VT_PERF_TEST(StructName, TestName)                  \
-  struct StructName##TestName : StructName {                \
-    StructName##TestName() {                                \
-      name_ = #TestName; }                                  \
-    void SetUp() override { StructName::SetUp(); }          \
-    void TearDown() override { StructName::TearDown(); }    \
-    void TestFunc() override;                               \
-  };                                                        \
-                                                            \
-  static struct StructName##TestName##_registerer_t {       \
-    StructName##TestName##_registerer_t() {                 \
-      PerfTestRegistry::AddTest(new StructName##TestName());\
-    }                                                       \
-  } StructName##TestName##_registerer;                      \
+#define VT_PERF_TEST(StructName, TestName)                                 \
+  struct StructName##TestName : StructName {                               \
+    StructName##TestName() {                                               \
+      name_ = #TestName;                                                   \
+    }                                                                      \
+    void SetUp() override {                                                \
+      StructName::SetUp();                                                 \
+    }                                                                      \
+    void TearDown() override {                                             \
+      StructName::TearDown();                                              \
+    }                                                                      \
+    void TestFunc() override;                                              \
+  };                                                                       \
+                                                                           \
+  static struct StructName##TestName##_registerer_t {                      \
+    StructName##TestName##_registerer_t() {                                \
+      PerfTestRegistry::AddTest(std::make_unique<StructName##TestName>()); \
+    }                                                                      \
+  } StructName##TestName##_registerer;                                     \
   void StructName##TestName::TestFunc()
 
-
 #define VT_PERF_TEST_MAIN()                                                      \
   int main(int argc, char** argv) {                                              \
     using namespace vt::tests::perf::common;                                     \
     MPI_Init(&argc, &argv);                                                      \
     int rank;                                                                    \
     MPI_Comm_rank(MPI_COMM_WORLD, &rank);                                        \
     for (const auto& test_base : PerfTestRegistry::GetTests()) {                 \
-      auto* test = dynamic_cast<PerfTestHarness*>(test_base);                    \
+      auto* test = dynamic_cast<PerfTestHarness*>(test_base.get());              \
       test->Initialize(argc, argv);                                              \
       auto const num_runs = test->GetNumRuns();                                  \
       StopWatch timer;                                                           \