Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a benchmark for repeated floats (packed and unpacked). #277

Merged
merged 1 commit into from
Dec 13, 2018

Conversation

judah
Copy link
Collaborator

@judah judah commented Dec 13, 2018

Also drop the size of the benchmark from 10k to 1k elements. From my testing,
the larger size isn't necessary to get meaningful results.


This change is Reviewable

Also drop the size of the benchmark from 10k to 1k elements.  From my testing,
the larger size isn't necessary to get meaningful results.
@judah judah requested a review from blackgnezdo December 13, 2018 01:09
judah added a commit that referenced this pull request Dec 13, 2018
Previously they were implemented by a loop that didn't optimize well.

This speeds up the benchmark from #277 (decoding repeated floats)
by a factor of 2.6x.  Specifically, the `float-packed/decode` and `float-unpacked/decode` benchmarks dropped from ~225us to ~85us.
judah added a commit that referenced this pull request Dec 13, 2018
Previously they were implemented by a loop that didn't optimize well.

This speeds up the benchmark from #277 (decoding repeated floats)
by a factor of 2.6x.  Specifically, the `float-packed/decode` and `float-unpacked/decode` benchmarks dropped from ~225us to ~85us.
@blackgnezdo blackgnezdo merged commit 1f2ede2 into master Dec 13, 2018
blackgnezdo pushed a commit that referenced this pull request Dec 13, 2018
Previously they were implemented by a loop that didn't optimize well.

This speeds up the benchmark from #277 (decoding repeated floats)
by a factor of 2.6x.  Specifically, the `float-packed/decode` and `float-unpacked/decode` benchmarks dropped from ~225us to ~85us.
@judah judah deleted the float-packing-bench branch December 13, 2018 06:14
judah added a commit that referenced this pull request Dec 19, 2018
Typos in #277; the benchmark names and proto types didn't match up.
blackgnezdo pushed a commit that referenced this pull request Dec 20, 2018
Typos in #277; the benchmark names and proto types didn't match up.
ylecornec pushed a commit to ylecornec/proto-lens that referenced this pull request Feb 19, 2024
Previously they were implemented by a loop that didn't optimize well.

This speeds up the benchmark from google#277 (decoding repeated floats)
by a factor of 2.6x.  Specifically, the `float-packed/decode` and `float-unpacked/decode` benchmarks dropped from ~225us to ~85us.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants