From a008bf82f4aa5ad6605622040c7b2fe7512dc0c7 Mon Sep 17 00:00:00 2001 From: dominic <510002+dmah42@users.noreply.github.com> Date: Tue, 13 Aug 2024 18:12:02 +0100 Subject: [PATCH] Ensure reported Time is walltime by removing spurious scaling by threads (#1836) * change the default to not scale --- src/benchmark_runner.cc | 6 ------ 1 file changed, 6 deletions(-) diff --git a/src/benchmark_runner.cc b/src/benchmark_runner.cc index 19f468af..a3809393 100644 --- a/src/benchmark_runner.cc +++ b/src/benchmark_runner.cc @@ -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";