#2380: metrics: Minor fixes

DARMA-tasking · Jan 15, 2025 · 638b412 · 638b412
1 parent b533dda
commit 638b412
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/vt/metrics/perf_data.cc b/src/vt/metrics/perf_data.cc
@@ -64,15 +64,15 @@ 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);
     }
   }
 
   // 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
@@ -91,7 +91,7 @@ struct PerfData: runtime::component::Component<PerfData>
    * Cleans up resources, closing file descriptors associated with performance
    * counters.
    */
-  ~PerfData();
+  virtual ~PerfData();
 
   /**
    * \brief Start performance measurement for a task