diff options
| author | Niklas Haas <git@haasn.dev> | 2025-03-31 14:40:16 +0200 |
|---|---|---|
| committer | Niklas Haas <git@haasn.dev> | 2025-03-31 15:27:24 +0200 |
| commit | 256a38101fd1bc26b020b870a202fd638616a0a6 (patch) | |
| tree | 8de4f88591fa04b6998f60fb14184a0705eb8835 /tests/checkasm/checkasm.h | |
| parent | 1015ea2ba15de8e7d4460496398c3eba6a42886d (diff) | |
| download | ffmpeg-256a38101fd1bc26b020b870a202fd638616a0a6.tar.gz | |
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
Diffstat (limited to 'tests/checkasm/checkasm.h')
| -rw-r--r-- | tests/checkasm/checkasm.h | 4 |
1 files 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) |