LaurentMazare / tensorflow-ocaml

OCaml bindings for TensorFlow
Apache License 2.0
283 stars 25 forks source link

Use Cstubs rather than libffi #15

Closed LaurentMazare closed 7 years ago

LaurentMazare commented 7 years ago

Switch the ctypes bindings to use cstubs rather than foreign/libffi.

braibant commented 7 years ago

This seems good, but I think you should update the opam file. IIRC, you should be able to remove the ctypes-foreign.

LaurentMazare commented 7 years ago

I would indeed like to get rid of the ctypes-foreign dependency, however I still use it for the coercion to static_funptr and I'm not sure how to avoid this.