On Wed, 23 Oct 2024, Martin Storsjö wrote:

This corresponds to commit 9278a14cf406f8edb5052c42b83750112bf5b515
in dav1d.

Omitting the C-only functions doesn't speed up benchmarking
anyway (as those has to be benchmarked before we know if we have
any corresponding assembly functions), and being able to benchmark
those functions without corresponding assembly can be valuable in
a number of cases.
---
tests/checkasm/checkasm.c | 48 +++++++++++++++++++--------------------
1 file changed, 23 insertions(+), 25 deletions(-)

Will push soon.

The only concern about this change I heard was from Ramiro on irc, who wasn't familiar with how to use the --bench=<pattern> parameter for limiting the set of functions to actually benchmark.

(This patch doesn't cause checkasm to benchmark more than it did before, it just prints the numbers that already were benchmarked.)

I'll also go ahead and backport this to the 7.1 branch. It's obviously not a regression fix or anything like that, but the change adds value for being able to use checkasm to benchmark compiler performance.

// Martin
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to