Skip to content

[v18.x backport] backport utf8 performance improvements #45650

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.

Sign up for GitHub

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

Closed
anonrig wants to merge 6 commits into from
Closed

[v18.x backport] backport utf8 performance improvements #45650

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
367d631
util: improve textdecoder decode performance
anonrig Nov 6, 2022
f5e3920
benchmark: fix text-decoder benchmark
anonrig Nov 7, 2022
22e9f5b
benchmark: add parameters to text-decoder benchmark
anonrig Nov 7, 2022
26fbc98
util: improve text-decoder performance
anonrig Nov 7, 2022
8dea4eb
util: improve text decoder performance
anonrig Nov 9, 2022
d86c37f
util: add fast path for utf8 encoding
anonrig Nov 12, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
benchmark: fix text-decoder benchmark 
PR-URL: #45363
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Rich Trott <rtrott@gmail.com>
  • Loading branch information
@anonrig
anonrig committed Nov 28, 2022
commit f5e3920aba1428f6e8260d039ed5ffa227d5f40c
4 changes: 3 additions & 1 deletion benchmark/util/text-decoder.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,8 @@ function main({ encoding, len, n, ignoreBOM }) {
const decoder = new TextDecoder(encoding, { ignoreBOM });

bench.start();
decoder.decode(buf);
for (let i = 0; i < n; i++) {
decoder.decode(buf);
}
bench.end(n);
}