8000 arm64 - switch to C stack before raising stack overflow by TheNumbat · Pull Request #4104 · oxcaml/oxcaml · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

arm64 - switch to C stack before raising stack overflow #4104

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

Jump to bottom
Merged
merged 2 commits into from
Jun 25, 2025
Merged

arm64 - switch to C stack before raising stack overflow #4104

merged 2 commits into from
Jun 25, 2025

Conversation

TheNumbat
Copy link
Contributor
@TheNumbat TheNumbat commented Jun 6, 2025 •
edited
Loading

Updates the arm64 version of caml_call_realloc_stack to be consistent with the amd64 version as of #4078

  • Switch back to the C stack before calling caml_raise_async
  • Don't bother restoring all regs, just replace the gc_regs bucket

@TheNumbat TheNumbat requested a review from mshinwell June 6, 2025 16:27
@TheNumbat TheNumbat added bug Something isn't working runtime labels Jun 6, 2025
@mshinwell mshinwell merged commit ce77ab0 into main Jun 25, 2025
32 checks passed
@mshinwell mshinwell deleted the arm64-try-realloc-stack branch June 25, 2025 09:42
mshinwell pushed a commit that referenced this pull request Jun 25, 2025
@TheNumbat @mshinwell
arm64 - switch to C stack before raising stack overflow (#4104)
d1e1622 
(cherry picked from commit ce77ab0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mshinwell mshinwell mshinwell approved these changes

Assignees
No one assigned
Labels
arm64 bug Something isn't working runtime
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@TheNumbat @mshinwell
0