report: Fix profile count aggregation (#707)

AddProfileEntries was creating a copy from the supplied update rather than the global store of counts. Signed-off-by: Charlie Egan <[email protected]>
StyraInc · May 8, 2024 · d826f64 · d826f64
1 parent 2bda636
commit d826f64
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/pkg/report/report.go b/pkg/report/report.go
@@ -91,12 +91,11 @@ func (r *Report) AddProfileEntries(prof map[string]ProfileEntry) {
  if _, ok := r.AggregateProfile[loc]; !ok {
  r.AggregateProfile[loc] = entry
  } else {
- profCopy := prof[loc]
+ profCopy := r.AggregateProfile[loc]
  profCopy.NumEval += entry.NumEval
  profCopy.NumRedo += entry.NumRedo
  profCopy.NumGenExpr += entry.NumGenExpr
  profCopy.TotalTimeNs += entry.TotalTimeNs
- profCopy.Location = entry.Location
  r.AggregateProfile[loc] = profCopy
  }
  }