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

[sc 453] don t stack overflow on very large terms #215

Merged
merged 7 commits into from
Feb 29, 2024

Conversation

LunaAmora
Copy link
Contributor

the following program works when running normally, with and without -Oall, with and without -L

MakeLongStr 0   acc = acc
MakeLongStr 1+p acc = (String.cons '*' (MakeLongStr p acc))

main = (MakeLongStr 16339 String.nil)

But putting it on the run_files golden tests it always hangs (lazy mode) or OOMs (strict mode) inside HVM-Core

@LunaAmora LunaAmora self-assigned this Feb 29, 2024
Copy link

This pull request has been linked to Shortcut Story #453: Don't stack overflow on very large terms.

@LunaAmora LunaAmora force-pushed the bug/sc-453/don-t-stack-overflow-on-very-large-terms branch from a9fe81b to 86d3d4d Compare February 29, 2024 16:20
@LunaAmora LunaAmora merged commit 00f1a77 into main Feb 29, 2024
5 checks passed
@LunaAmora LunaAmora deleted the bug/sc-453/don-t-stack-overflow-on-very-large-terms branch February 29, 2024 17:59
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.

3 participants