Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Better inlining ? #357

Closed
dbuenzli opened this issue Aug 18, 2015 · 3 comments · Fixed by #1584
Closed

Better inlining ? #357

dbuenzli opened this issue Aug 18, 2015 · 3 comments · Fixed by #1584
Labels
enhancement

Comments

@dbuenzli
Copy link
Contributor

Somehow it is annoying that self-recursive functions that go through a simple error monad do blow the stack. See for example here dbuenzli/otfm@4ea6a34
@hhugo
Copy link
Member

hhugo commented Aug 22, 2015

#154

@hhugo hhugo mentioned this issue Jan 26, 2016
Closed
@ejgallego ejgallego mentioned this issue Mar 4, 2016
Open
@hhugo hhugo mentioned this issue Mar 28, 2016
Closed
5 tasks
@hhugo hhugo mentioned this issue Mar 10, 2024
Merged
@hhugo
Copy link
Member

hhugo commented Mar 10, 2024

@dbuenzli, 9 years later .. would you want to give #1584 a try ?

@dbuenzli
Copy link
Contributor Author

I'm afraid I don't have a test case at hand. But thanks for the work !

# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Compiler: Rework inlining ocsigen/js_of_ocaml
2 participants
@dbuenzli @hhugo