Skip to content

Shrink JsStr from 24 to 16 bytes#4322

Merged
jedel1043 merged 2 commits intomainfrom
optimization/small-jsstr
Jul 5, 2025
Merged

Shrink JsStr from 24 to 16 bytes#4322
jedel1043 merged 2 commits intomainfrom
optimization/small-jsstr

Conversation

@HalidOdat
Copy link
Member

This PR makes the JsStr's len be encoded with TaggedLen reducing the size from 24 to 16 bytes, shaves 5464 bytes from the const RAW_STRINGS array.

The tests have been run under miri and it's happy with the change 🥳


main

PROGRESS Richards
RESULT Richards 144
PROGRESS DeltaBlue
RESULT DeltaBlue 151
PROGRESS Encrypt
PROGRESS Decrypt
RESULT Crypto 144
PROGRESS RayTrace
RESULT RayTrace 333
PROGRESS Earley
PROGRESS Boyer
RESULT EarleyBoyer 385
PROGRESS RegExp
RESULT RegExp 72.1
PROGRESS Splay
RESULT Splay 527
PROGRESS NavierStokes
RESULT NavierStokes 334
SCORE 218

PR

PROGRESS Richards
RESULT Richards 146
PROGRESS DeltaBlue
RESULT DeltaBlue 152
PROGRESS Encrypt
PROGRESS Decrypt
RESULT Crypto 146
PROGRESS RayTrace
RESULT RayTrace 344
PROGRESS Earley
PROGRESS Boyer
RESULT EarleyBoyer 403
PROGRESS RegExp
RESULT RegExp 74.5
PROGRESS Splay
RESULT Splay 534
PROGRESS NavierStokes
RESULT NavierStokes 348
SCORE 224
undefined

@github-actions
Copy link

github-actions bot commented Jul 5, 2025

Test262 conformance changes

Test result main count PR count difference
Total 50,595 50,595 0
Passed 47,238 47,238 0
Ignored 1,964 1,964 0
Failed 1,393 1,393 0
Panics 0 0 0
Conformance 93.36% 93.36% 0.00%

@jedel1043 jedel1043 added this pull request to the merge queue Jul 5, 2025
Merged via the queue into main with commit 61f5b02 Jul 5, 2025
14 checks passed
@jedel1043 jedel1043 deleted the optimization/small-jsstr branch July 5, 2025 20:53
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