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

sea: don't set code cache flags when snapshot is used #54120

Merged
merged 1 commit into from
Aug 5, 2024

Commits on Jul 30, 2024

  1. sea: don't set code cache flags when snapshot is used

    When both useCodeCache and useSnapshot are set, we generate the
    snapshot and skip the generation of the code cache since the
    snapshot already includes the code cache. But we previously still
    persist the code cache setting in the flags that got serialized
    into the SEA, so the resulting executable would still try to read
    the code cache even if it's not added to the SEA, leading to a flaky
    crash caused by OOB on some platforms.
    
    This patch fixes the crash by ignoring the code cache setting when
    generating the flag if both snapshot and code cache is configured.
    joyeecheung committed Jul 30, 2024
    Configuration menu
    Copy the full SHA
    d6ed160 View commit details
    Browse the repository at this point in the history