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

macOS: print stackoverflow warning if the overflow happened inside GC #55211

Merged
merged 1 commit into from
Jul 22, 2024

Conversation

d-netto
Copy link
Member

@d-netto d-netto commented Jul 22, 2024

See discussion in #55192.

I'm not sure how easy it is to recover from a stack overflow inside the GC (or whether we should even try to do it), but we should at least print a stack overflow warning here IMO.

@d-netto d-netto added GC Garbage collector error messages Better, more actionable error messages labels Jul 22, 2024
@d-netto d-netto requested review from vtjnash and gbaraldi July 22, 2024 18:59
@vtjnash vtjnash added the merge me PR is reviewed. Merge when all tests are passing label Jul 22, 2024
@d-netto d-netto merged commit b75c133 into master Jul 22, 2024
9 of 10 checks passed
@d-netto d-netto deleted the dcn-macos-stack-overflow branch July 22, 2024 23:01
@d-netto d-netto removed the merge me PR is reviewed. Merge when all tests are passing label Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
error messages Better, more actionable error messages GC Garbage collector
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants