Skip to content

Commit

Permalink
Fix - Removes redundant TotalBatchesLen from `ThreadExecuteTimings:…
Browse files Browse the repository at this point in the history
…:accumulate()` (#34179)

Removes redundant TotalBatchesLen from ThreadExecuteTimings::accumulate().

(cherry picked from commit c0a4fc8)
  • Loading branch information
Lichtso authored and mergify[bot] committed Dec 6, 2023
1 parent 311e986 commit 4de56c3
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 4de56c3

Please sign in to comment.