1
0
mirror of https://github.com/google/benchmark.git synced 2025-04-29 22:40:33 +08:00

perf_counters_test.cc: Don't divide by iterations

Perf counters are now divided by iterations, so dividing again
in the test is wrong.
This commit is contained in:
Jesse Rosenstock 2023-08-21 13:33:42 +02:00 committed by GitHub
parent 25cd9818b8
commit f112198190
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -67,22 +67,17 @@ static void CheckSimple(Results const& e) {
double withoutPauseResumeInstrCount = 0.0;
double withPauseResumeInstrCount = 0.0;
static void CheckInstrCount(double* counter, Results const& e) {
BM_CHECK_GT(e.NumIterations(), 0);
*counter = e.GetAs<double>("INSTRUCTIONS") / e.NumIterations();
static void SaveInstrCountWithoutResume(Results const& e) {
withoutPauseResumeInstrCount = e.GetAs<double>("INSTRUCTIONS");
}
static void CheckInstrCountWithoutResume(Results const& e) {
CheckInstrCount(&withoutPauseResumeInstrCount, e);
}
static void CheckInstrCountWithResume(Results const& e) {
CheckInstrCount(&withPauseResumeInstrCount, e);
static void SaveInstrCountWithResume(Results const& e) {
withPauseResumeInstrCount = e.GetAs<double>("INSTRUCTIONS");
}
CHECK_BENCHMARK_RESULTS("BM_Simple", &CheckSimple);
CHECK_BENCHMARK_RESULTS("BM_WithoutPauseResume", &CheckInstrCountWithoutResume);
CHECK_BENCHMARK_RESULTS("BM_WithPauseResume", &CheckInstrCountWithResume);
CHECK_BENCHMARK_RESULTS("BM_WithoutPauseResume", &SaveInstrCountWithoutResume);
CHECK_BENCHMARK_RESULTS("BM_WithPauseResume", &SaveInstrCountWithResume);
int main(int argc, char* argv[]) {
if (!benchmark::internal::PerfCounters::kSupported) {