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.

Sign up for GitHub

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

Jump to bottom

Split JSPI and Cps effect runtime to improve readability #1788

Open

hhugo opened this issue Jan 6, 2025 · 0 comments

Assignees

Labels

Member

hhugo commented Jan 6, 2025

They currently both live in effect.wat with similar names.

caml_alloc_stack vs caml_cps_alloc_stack
%perform vs caml_perform_effect
...

We could move jspi and cps part in their own file together with their helpers.
@vouillon, what do you think ?

I also wonder if making names more symmetric could help

hhugo added the enhancement label

hhugo assigned vouillon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment