KC Sivaramakrishnan

Results 217 comments of KC Sivaramakrishnan
trafficstars

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.