Use js_of_ocaml's --enable=effects flag to fix stackoverflows
Motivation and Context
We're facing stackoverflows around LexerLib
Description
This MR uses js_of_ocaml's new --enable=effects
flag which turns non tail recursive code to tail recursive CPS (therefore fixing stackoverflows in browser)
Component
-
compiler -
website -
webide -
vscode-plugin -
debugger
Types of changes
-
Bug fix (non-breaking change which fixes an issue) -
New feature (non-breaking change which adds functionality) -
Breaking change (fix or feature that would cause existing functionality to not work as expected) -
Performance improvement (non-breaking change that improves performance) -
None (change with no changelog)
Changelog
Fixes stackoverflows during Lexing in the browser
Checklist:
-
Changes follow the existing coding style (use dune @fmt
to check).
Merge request reports
Activity
requested review from @ulrikstrid
mentioned in commit 4e2420c1
added 24 commits
-
79598de4...b26c9207 - 22 commits from branch
dev
- 4e2420c1 - Merge branch 'prometheansacrifice@jsoo-use-enable-effects-flag' into 'dev'
- 14a39f0c - [Bot] add changelog entry
-
79598de4...b26c9207 - 22 commits from branch
A new deploy preview is available on Netlify at https://2417-c554f9b0--ligolang.netlify.app, you can view the logs at https://app.netlify.com/sites/ligolang/deploys/64080872b1ae2452bd0520e8
Edited by Ligo Langmentioned in commit d43fd9d7
Please register or sign in to reply