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

Cache property enumeration in JSON.stringify like we do for Object.assign #4907

Merged

Conversation

sethbrenith
Copy link
Contributor

JSON payloads tend to be relatively repetitive, so we can save a lot on large serializations by reusing object property iteration. On a microbenchmark that serializes and parses a repetitive piece of data that I found in Speedometer, I see a 35% improvement.

@sethbrenith
Copy link
Contributor Author

@dotnet-bot
test OSX _no_jit_shared_osx_osx_test please
test OSX static_osx_osx_debug please

@sethbrenith
Copy link
Contributor Author

@dotnet-bot test OSX static_osx_osx_debug please

@chakrabot chakrabot merged commit 8b2536f into chakra-core:master Apr 2, 2018
chakrabot pushed a commit that referenced this pull request Apr 2, 2018
…fy like we do for Object.assign

Merge pull request #4907 from sethbrenith:user/sethb/stringify-cache-iter

JSON payloads tend to be relatively repetitive, so we can save a lot on large serializations by reusing object property iteration. On a microbenchmark that serializes and parses a repetitive piece of data that I found in Speedometer, I see a 35% improvement.
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.

4 participants