-
Notifications
You must be signed in to change notification settings - Fork 184
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
Compiler: demonstrate and fix bug from 1354 #1356
Conversation
This works much better than I would have thought. With the right intuition (this is just argument passing), the changes to the optimization phases are straightforward. The complexity is localized in |
This is cleaner since, when we have an 'Assign (x, y)', x is in scope.
Thanks for the fix. I've squashed some commit. Do you have more on this PR ? Can we merge ? |
I think we can merge |
…s_of_ocaml-ppx_deriving_json, js_of_ocaml-ppx, js_of_ocaml-lwt and js_of_ocaml-compiler (5.0.0) CHANGES: ## Features/Changes * Compiler: add support for effect handlers (--enable=effects) * Compiler: small refactoring in code generation * Compiler: check build info compatibility when linking js file. * Misc: fix and update benchmarks * Misc: upgrade CI * Toplevel: recover more names when generating code during toplevel evaluation * Runtime: wrapping exception or not is now controled in the runtime. ## Bug fixes * Runime: Gc.finalise_last should not be eliminated * Tyxml: reactive dom needed a fix after ocsigen/js_of_ocaml#1268 (ocsigen/js_of_ocaml#1353) * Toplevel: Make sure the toplevel uses the correct memory representaion for strings * Compiler: fix minifier, missing constraint on try-catch blocks. * Compiler: Miscompilation of code involving references and exceptions (ocsigen/js_of_ocaml#1354, ocsigen/js_of_ocaml#1356)
continues/replaces #1355
fix #1354