unison icon indicating copy to clipboard operation
unison copied to clipboard

Add Text benchmarks

Open satotake opened this issue 3 years ago • 2 comments

Overview

I am trying #2599. I am not sure what is the focuses here. So, as a starting point, I added basic benchmarks for Unison.Util.Text vs. Data.Text. If this issue does not target Data.Text, I will remove them. And I have not done Bytes or Rope yet.

  • It is not so helpful to increase n here. It might be good to limit the patterns.
  • Some operations of Text underlying Rope (size, at) are faster than Data.Text

Interesting/controversial decisions

I ignored parser-typechecker/benchmarks/runtime/Main.hs tentatively. It was not updated and I tried to fix the regression. But I gave it up when I found that we need to use CombIx in Let.


This change is Reviewable

satotake avatar Nov 14 '21 13:11 satotake

@satotake We have #2631 to fix benchmarks/runtime/Main.hs, sorry about that.

aryairani avatar Nov 14 '21 23:11 aryairani

@aryairani no worries, sorry for missing that.

satotake avatar Nov 15 '21 12:11 satotake