search

art-w / sherlodoc

Fuzzy type search for OCaml documentation
MIT License
73 stars 6 forks source link

Benchmark with `wasm_of_ocaml` #28

Open art-w opened 8 months ago

art-w commented 8 months ago

As suggested by @rikusilvola, we should see huge improvements by compiling the search algorithms with wasm_of_ocaml since it's pure computation! However we don't have benchmarks to evaluate the impact of the change atm.

OlivierNicole commented 8 months ago

Indeed you should probably see an improvement on pure computation!

IIRC Chrome has a very good profiler built-in. Compiling to wasm should not be hard once you have a JS bundle of your application. (We also have a fork of Dune to add the wasm target even more easily.)

EmileTrotignon commented 8 months ago

This should out of the box with the right version of brr, but it will probably result in larger code size according to Jerome Vouillon.