[PATCH] D99471: [OCaml] Fix unsafe uses of Store_field

Vaivaswatha Nagaraj via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 29 09:53:56 PDT 2021


vaivaswatha accepted this revision.
vaivaswatha added a comment.
This revision is now accepted and ready to land.

This fixes a potentially hard-to-discover bug ! The last thing that a user of the OCaml API would think of is "oh this function uses `Store_field` to initialize fields of blocks allocated with `caml_alloc_small` and that could lead to a hard to trace GC bug".

Thank you !.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D99471/new/

https://reviews.llvm.org/D99471



More information about the llvm-commits mailing list