Skip to content

Commit

Permalink
Ensure reported Time is walltime by removing spurious scaling by thre…
Browse files Browse the repository at this point in the history
…ads (#1836)

* change the default to not scale
  • Loading branch information
dmah42 authored Aug 13, 2024
1 parent b884717 commit a008bf8
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions src/benchmark_runner.cc
Original file line number Diff line number Diff line change
Expand Up @@ -292,12 +292,6 @@ BenchmarkRunner::IterationResults BenchmarkRunner::DoNIterations() {
// And get rid of the manager.
manager.reset();

// Adjust real/manual time stats since they were reported per thread.
i.results.real_time_used /= b.threads();
i.results.manual_time_used /= b.threads();
// If we were measuring whole-process CPU usage, adjust the CPU time too.
if (b.measure_process_cpu_time()) i.results.cpu_time_used /= b.threads();

BM_VLOG(2) << "Ran in " << i.results.cpu_time_used << "/"
<< i.results.real_time_used << "\n";

Expand Down

0 comments on commit a008bf8

Please sign in to comment.