-
Notifications
You must be signed in to change notification settings - Fork 68
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
Make ephemerons immutable. #739
Conversation
This is a port of ocaml/ocaml#10737 for multicore.
The consensus is that we keep the |
The module is not concurrency safe. The idea is that after 5.00 MVP, ephemerons will be rewritten over Bucket interface.
@ctk21 this PR should be good to go except for a quick review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I took a (time limited) look. The PR does not alter the underlying runtime ephemeron GC mechanisms. However I think it does remove deprecated functions and makes the OCaml interface match up with post ocaml/ocaml#10737
I spotted a minor nit: the PR still references to the removed iter
, fold
functions in the documentation strings in ephemeron.ml
.
I did not deeply review the testsuite alterations.
I have further taken a look at the alterations to |
Thanks @ctk21. Removed mentions of |
Let me know if anything else looks odd.. |
LGTM, merging! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good except for a small hiccup in ephetest_par.ml
.
…ticore/new_ephe_api Make ephemerons immutable.
…ticore/new_ephe_api Make ephemerons immutable.
This is a port of ocaml/ocaml#10737 for multicore. Compared to ocaml/ocaml#10737 it removes all the deprecated functions.