From c0a4fc870c4d86fb15bbefe8515e642f93ea74e0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Alexander=20Mei=C3=9Fner?= Date: Mon, 20 Nov 2023 19:21:01 +0100 Subject: [PATCH] Fix - Removes redundant `TotalBatchesLen` from `ThreadExecuteTimings::accumulate()` (#34179) Removes redundant TotalBatchesLen from ThreadExecuteTimings::accumulate(). --- program-runtime/src/timings.rs | 7 ------- 1 file changed, 7 deletions(-) diff --git a/program-runtime/src/timings.rs b/program-runtime/src/timings.rs index 0e2e4956a..8eeb9c5a0 100644 --- a/program-runtime/src/timings.rs +++ b/program-runtime/src/timings.rs @@ -300,13 +300,6 @@ impl ThreadExecuteTimings { } pub fn accumulate(&mut self, other: &ThreadExecuteTimings) { - self.execute_timings.saturating_add_in_place( - ExecuteTimingType::TotalBatchesLen, - *other - .execute_timings - .metrics - .index(ExecuteTimingType::TotalBatchesLen), - ); self.execute_timings.accumulate(&other.execute_timings); saturating_add_assign!(self.total_thread_us, other.total_thread_us); saturating_add_assign!(