search

mirage / checkseum

MIT License
15 stars 13 forks source link

Optimize checkseum #64

Closed dinosaure closed 2 years ago

hannesm commented 2 years ago

is the addition of bench/main.html in this PR intentional?

dinosaure commented 2 years ago

is the addition of bench/main.html in this PR intentional?

I'm not sure about that. I would like to say yes because it's a standalone benchmark of checkseum which permits to show performances of it and I would like to say no because it's a huge file.

dinosaure commented 2 years ago

I finally deleted the bench/main.html and the compilation of the benchmark (which should be done via dune build --profile benchmark now).

dinosaure commented 2 years ago

I check by hands tests on 4.11.2+32bit, it should be fine on 32bit architectures.