Skip to content

Commit

Permalink
v1.17: Fix - Removes redundant TotalBatchesLen from `ThreadExecuteT…
Browse files Browse the repository at this point in the history
…imings::accumulate()` (backport of #34179) (#34347)

Fix - Removes redundant `TotalBatchesLen` from `ThreadExecuteTimings::accumulate()` (#34179)

Removes redundant TotalBatchesLen from ThreadExecuteTimings::accumulate().

(cherry picked from commit c0a4fc8)

Co-authored-by: Alexander Meißner <AlexanderMeissner@gmx.net>
  • Loading branch information
mergify[bot] and Lichtso authored Dec 7, 2023
1 parent 311e986 commit d17243f
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions program-runtime/src/timings.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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!(
Expand Down

0 comments on commit d17243f

Please sign in to comment.