[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