#1009: memory footprinting: do not use countBytes when | operator is …

…available
DARMA-tasking · Nov 24, 2020 · f424afb · f424afb
1 parent 5faf0bd
commit f424afb
Show file tree

Hide file tree

Showing 6 changed files with 14 additions and 19 deletions.
diff --git a/src/vt/topos/location/cache/cache.h b/src/vt/topos/location/cache/cache.h
@@ -78,9 +78,8 @@ struct LocationCache {
   template <typename Serializer>
   void serialize(Serializer& s) {
     s | cache_
-      | max_size_;
-
-    s.countBytes(lookup_);
+      | max_size_
+      | lookup_;
   }
 
  private:

diff --git a/src/vt/trace/trace_log.h b/src/vt/trace/trace_log.h
@@ -189,10 +189,10 @@ struct Log final {
       reset();
     }
 
-  template <typename Serializer>
-  void serialize(Serializer& s) {
-    s.countBytes(*this);
-  }
+    template <typename Serializer>
+    void serialize(Serializer& s) {
+      s.countBytes(*this);
+    }
 
   private:
 

diff --git a/src/vt/utils/container/concurrent_deque_locked.h b/src/vt/utils/container/concurrent_deque_locked.h
@@ -106,9 +106,8 @@ struct ConcurrentDequeLocked {
   template <typename Serializer>
   void serialize(Serializer& s) {
     s | needs_lock_
-      | container_;
-
-    s.countBytes(container_mutex_);
+      | container_
+      | container_mutex_;
   }
 
 private:

diff --git a/src/vt/utils/container/process_ready_buffer.h b/src/vt/utils/container/process_ready_buffer.h
@@ -87,9 +87,8 @@ struct ProcessBuffer {
     s | buffer_
       | needs_lock_
       | worker_
-      | process_fn_;
-
-    s.countBytes(mutex_);
+      | process_fn_
+      | mutex_;
   }
 
 private:

diff --git a/src/vt/worker/worker_group.h b/src/vt/worker/worker_group.h
@@ -100,9 +100,8 @@ struct WorkerGroupAny
   void serialize(SerializerT& s) {
     s | finished_fn_
       | initialized_
-      | num_workers_;
-
-    s.countBytes(workers_);
+      | num_workers_
+      | workers_;
   }
 
 private:

diff --git a/src/vt/worker/worker_group_omp.h b/src/vt/worker/worker_group_omp.h
@@ -103,9 +103,8 @@ struct WorkerGroupOMP
       | ready_
       | initialized_
       | num_workers_
-      | worker_state_;
-
-    s.countBytes(enqueue_worker_mutex_);
+      | worker_state_
+      | enqueue_worker_mutex_;
   }
 
 private: