Skip to content

Float the WriteUtf8V2 patch#5465

Merged
jasnell merged 1 commit intomainfrom
jasnell/float-writeutf8v2-patch
Nov 5, 2025
Merged

Float the WriteUtf8V2 patch#5465
jasnell merged 1 commit intomainfrom
jasnell/float-writeutf8v2-patch

Conversation

@jasnell
Copy link
Collaborator

@jasnell jasnell commented Nov 5, 2025

No description provided.

@jasnell jasnell requested review from anonrig and erikcorry November 5, 2025 20:37
@jasnell jasnell requested review from a team as code owners November 5, 2025 20:37
@codspeed-hq
Copy link

codspeed-hq bot commented Nov 5, 2025

CodSpeed Performance Report

Merging #5465 will improve performances by ×14

Comparing jasnell/float-writeutf8v2-patch (95a2c57) with main (746c8f0)

Summary

⚡ 6 improvements
✅ 28 untouched
⏩ 9 skipped1

Benchmarks breakdown

Benchmark BASE HEAD Change
EncodeInto_ASCII_1024[1/0/1024] 7.9 ms 2.6 ms ×3
EncodeInto_ASCII_256[1/0/256] 3.8 ms 2.5 ms +49.98%
EncodeInto_ASCII_8192[1/0/8192] 45.8 ms 3.2 ms ×14
Encode_ASCII_1024[0/0/1024] 8.9 ms 3.7 ms ×2.4
Encode_ASCII_256[0/0/256] 4.4 ms 3.1 ms +41.31%
Encode_ASCII_8192[0/0/8192] 55.7 ms 13.1 ms ×4.2

Footnotes

  1. 9 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@jasnell jasnell merged commit a4deb78 into main Nov 5, 2025
32 of 33 checks passed
@jasnell jasnell deleted the jasnell/float-writeutf8v2-patch branch November 5, 2025 23:30
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.

2 participants