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

Refactor JS parser visitor step into individual functions to reduce stack space usage #15986

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

Jarred-Sumner
Copy link
Collaborator

@Jarred-Sumner Jarred-Sumner commented Dec 25, 2024

What does this PR do?

Fixes #13233

Bun version visitAndAppendStmt stack space usage
This PR 1552 bytes
Bun v1.1. 42 7456 bytes

https://claude.ai/share/ef33dcc7-5ff7-425b-babe-f4370ff45855

How did you verify your code works?

@robobun
Copy link

robobun commented Dec 25, 2024

Updated 7:16 PM PT - Dec 24th, 2024

@Jarred-Sumner, your commit 9fd10a5 is building: #8488

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.

Transpiler stack overflow with many else if(0) branches
2 participants