[PATCH] D99475: [OCaml] Omit unnecessary GC root registrations

Josh Berdine via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 28 15:33:08 PDT 2021


jberdine created this revision.
jberdine added a reviewer: vaivaswatha.
Herald added a reviewer: whitequark.
jberdine requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

The current code does not follow the simple interface to the OCaml GC,
where GC roots are registered conservatively, only initializing
allocations are performed, etc. This is intentional, as stated in the
opening file comments. On the other hand, the current code does
register GC roots in many situations where it is not strictly
necessary. This diff omits many of them.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D99475

Files:
  llvm/bindings/ocaml/llvm/llvm_ocaml.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D99475.333742.patch
Type: text/x-patch
Size: 10855 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210328/ace27594/attachment.bin>


More information about the llvm-commits mailing list