Skip to content

Commit

Permalink
Merge pull request #12834 from gadmm/refactor_thread_register
Browse files Browse the repository at this point in the history
Refactor caml_c_thread_(un)register to fix various bugs
  • Loading branch information
gasche authored Jan 12, 2024
2 parents cedff58 + 97c2ee7 commit 5348fa9
Show file tree
Hide file tree
Showing 2 changed files with 125 additions and 97 deletions.
12 changes: 9 additions & 3 deletions Changes
Original file line number Diff line number Diff line change
Expand Up @@ -234,9 +234,9 @@ OCaml 5.2.0
Ojeda Bar)

- #11911, #12382, #12383: Restore statmemprof functionality in part
(backtrace buffers, per-thread and per-domain data
structures). (Nick Barnes, review by Gabriel Scherer, Fabrice Buoro,
Sadiq Jaffer, and Guillaume Munch-Maccagnoni).
(backtrace buffers, per-thread and per-domain data structures).
(Nick Barnes, review by Gabriel Scherer, Fabrice Buoro, Sadiq
Jaffer, and Guillaume Munch-Maccagnoni).

- #12735: Store both ends of the stack chain in continuations
(Leo White, review by Miod Vallat and KC Sivaramakrishnan)
Expand All @@ -260,6 +260,12 @@ OCaml 5.2.0
style that the compiler driver uses.
(David Allsopp, review by Gabriel Scherer)

- #12773, #12830, #12834: Rewrite `caml_c_thread_(un)register` to fix
various bugs.
(Guillaume Munch-Maccagnoni, reported by Miod Vallat, suggested by
Hari Hara Naveen S, reviewed by Fabrice Buoro, Gabriel Scherer and
Miod Vallat)

- #12839: Remove ATOMIC_UINTNAT_INIT from camlatomic.h (as part of a larger
cleanup of camlatomic.h)
(David Allsopp, review by Antonin Décimo, Sébastien Hinderer, Samuel Hym,
Expand Down
Loading

0 comments on commit 5348fa9

Please sign in to comment.