From 256a38101fd1bc26b020b870a202fd638616a0a6 Mon Sep 17 00:00:00 2001 From: Niklas Haas Date: Mon, 31 Mar 2025 14:40:16 +0200 Subject: [PATCH] tests/checkasm: fix wrong summation of bench time This was changed 8 years ago with the introduction of the linux-perf path, with seemingly no justification at the time. Likely a developer oversight from testing. This bug not only made --runs completely ineffective, but also meant that we didn't actually correctly filter out outliers. Fixes: e0d56f097f42bcdbe6c3b2f57df62a4da63f2094 --- tests/checkasm/checkasm.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/checkasm/checkasm.h b/tests/checkasm/checkasm.h index 75c81317ba..131e71c56e 100644 --- a/tests/checkasm/checkasm.h +++ b/tests/checkasm/checkasm.h @@ -364,8 +364,8 @@ typedef struct CheckasmPerf { }\ }\ emms_c();\ - perf->cycles += t;\ - perf->iterations++;\ + perf->cycles += tsum;\ + perf->iterations += tcount;\ checkasm_set_signal_handler_state(0);\ }\ } while (0)