[PATCH] OCaml bindings: make DataLayout.t automatically managed

Peter Zotov whitequark at whitequark.org
Sat Oct 12 11:18:13 PDT 2013


Hi sylvestre.ledru,

Since DataLayout does not own any objects which can be referenced from OCaml heap, it is safe to finalize it from OCaml's GC.

http://llvm-reviews.chandlerc.com/D1926

Files:
  bindings/ocaml/executionengine/executionengine_ocaml.c
  bindings/ocaml/executionengine/llvm_executionengine.ml
  bindings/ocaml/target/llvm_target.ml
  bindings/ocaml/target/llvm_target.mli
  bindings/ocaml/target/target_ocaml.c
  test/Bindings/Ocaml/ipo_opts.ml
  test/Bindings/Ocaml/scalar_opts.ml
  test/Bindings/Ocaml/target.ml
  test/Bindings/Ocaml/vectorize_opts.ml
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D1926.1.patch
Type: text/x-patch
Size: 11989 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131012/82aba7d2/attachment.bin>


More information about the llvm-commits mailing list