Skip to content

feat: Criterion Benchmark#185

Open
GunterSchmidt wants to merge 3 commits intouutils:mainfrom
GunterSchmidt:cmp-feat--criterion-benchmarks
Open

feat: Criterion Benchmark#185
GunterSchmidt wants to merge 3 commits intouutils:mainfrom
GunterSchmidt:cmp-feat--criterion-benchmarks

Conversation

@GunterSchmidt
Copy link

During development it is often helpful to be able to quickly benchmark the recent changes.

Criterion offers the ability to bench single functions.

This PR introduces criterion and has benches for the parsers (negligible) and cmp compare.

Has also a support function to generate large files to compare for bench-marking.

@sylvestre
Copy link
Collaborator

we, uutils, use codspeed

ex: uutils/coreutils#10996

could you please copy what is done with coreutils ?
thanks

@oech3
Copy link
Contributor

oech3 commented Mar 2, 2026

CodSpeed does not count some syscalls. How about your suggestion?

@GunterSchmidt
Copy link
Author

@sylvestre : I am not sure what you mean by "copy". Could you please elaborate.

I just migrated to codspeed criterion drop-in replacement. But would you like me to drop Criterion and use the divan benchmark instead? At this early stage this would not be too hard.

@sylvestre
Copy link
Collaborator

yes, please use divan
and i will connect codspeed to this repo

@GunterSchmidt
Copy link
Author

I removed Criterion and migrated the tests to Divan.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants