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

Vaivaswatha Nagaraj via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Apr 3 02:23:19 PDT 2021


vaivaswatha added a comment.

In D99475#2663806 <https://reviews.llvm.org/D99475#2663806>, @jberdine wrote:

> See https://github.com/jberdine/llvm-project/tree/ocaml for this stack on github.
>
> For the init code, I just mean to add that `Gc.set` call somewhere in your code that uses the LLVM bindings before you do much with Llvm.

@jberdine I built your branch locally and linked to it from my project that uses LLVM (and its OCaml bindings). I added the call to `Gc.set` before calling any of the LLVM functions. Everything worked fine and my testsuite succeeded. Let me know if you want me to try out anything else. Thank you.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99475



More information about the llvm-commits mailing list