From 638b4123501a4c8f2e9560ff6145225ecaddb1f3 Mon Sep 17 00:00:00 2001 From: Pierre Pebay Date: Wed, 15 Jan 2025 17:19:54 +0100 Subject: [PATCH] #2380: metrics: Minor fixes --- src/vt/metrics/perf_data.cc | 4 ++-- src/vt/metrics/perf_data.h | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/vt/metrics/perf_data.cc b/src/vt/metrics/perf_data.cc index 5b5abd141f..37e41512c4 100644 --- a/src/vt/metrics/perf_data.cc +++ b/src/vt/metrics/perf_data.cc @@ -64,7 +64,7 @@ PerfData::PerfData() } } - for (const auto &event_name : event_names_) { + for (auto const& event_name : event_names_) { if (event_map_.find(event_name) == event_map_.end()) { cleanupBeforeAbort(); vtAbort("Event name isn't in known perf events map: " + event_name); @@ -72,7 +72,7 @@ PerfData::PerfData() } // Initialize perf events once and store file descriptors - for (const auto &event_name : event_names_) { + for (auto const& event_name : event_names_) { struct perf_event_attr pe = {}; pe.type = event_map_.at(event_name).first; pe.size = sizeof(struct perf_event_attr); diff --git a/src/vt/metrics/perf_data.h b/src/vt/metrics/perf_data.h index 197a73a6bc..a3e1792e4a 100644 --- a/src/vt/metrics/perf_data.h +++ b/src/vt/metrics/perf_data.h @@ -91,7 +91,7 @@ struct PerfData: runtime::component::Component * Cleans up resources, closing file descriptors associated with performance * counters. */ - ~PerfData(); + virtual ~PerfData(); /** * \brief Start performance measurement for a task