Skip to content
Snippets Groups Projects

Use js_of_ocaml's --enable=effects flag to fix stackoverflows

Merged prometheansacrifice requested to merge prometheansacrifice@jsoo-use-enable-effects-flag into dev

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

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading