Skip to content

Commit

Permalink
[perf2bolt] Fix print report for pre-aggregated profile
Browse files Browse the repository at this point in the history
Summary:
For pre-aggregated profile, we were using the number of records in the
profile for `NumTraces` ignoring the counts per record. As a result,
the reported percentage of mismatched traces was bogus.

(cherry picked from FBD15093123)
  • Loading branch information
maksfb committed Apr 25, 2019
1 parent 492e4a5 commit 5717b0c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion bolt/src/DataAggregator.cpp
Expand Up @@ -1407,7 +1407,7 @@ void DataAggregator::processPreAggregated() {
AggrEntry.From.Offset, false};
LBREntry Second{AggrEntry.To.Offset, AggrEntry.To.Offset, false};
doTrace(First, Second, AggrEntry.Count);
++NumTraces;
NumTraces += AggrEntry.Count;
break;
}
}
Expand Down

0 comments on commit 5717b0c

Please sign in to comment.