KC Sivaramakrishnan
KC Sivaramakrishnan
Thanks @jonludlam for the help. With the latest commits, the outstanding tasks mentioned in https://github.com/ocaml/ocaml/pull/11423#issuecomment-1182797192 are complete.
@xavierleroy > Camlinternal* modules because they complicate the structure of the stdlib I needed to introduce the `CamlInternalEffect` module to break the cyclic dependency between the `Printexc` and `Effect` modules....
There are a number of issues with the current implementation. ## Exception names not registered The following program crashes since exception names may not registered. This was pointed out by...
@Octachron the commit looks good to me. It would be great if you can rebase this change and push the PR towards a merge. I'm unfortunately quite busy before ICFP....
The PR as it stands looks good to me. The only point of contention is the choice between the strings `"Unhandled effect: %s"` and `"Stdlib.Effect.Unhandled(%s)"`. This can be changed in...
Thanks @ejgallego! We shall use this for our experiments.
I have now included everything that I wanted to say in the parallelism chapter. CC @gasche @fpottier.
@fpottier I've pushed the rendered page. It should go live in a few minutes. Thanks for taking a look.
Thanks for the reviews so far. For reviewers who have write permission to `ocaml/ocaml`, please feel free to directly push minor edits to `kayceesrk:domains_manual` branch.
Thanks @fabbing for having a look.