search

mmtk / mmtk-julia

Julia binding for MMTk
13 stars 9 forks source link

Use build script to run bindgen #88

Open qinsoon opened 1 year ago

qinsoon commented 1 year ago

Currently we generate a Rust version for mmtk_julia_types.h using rust-bindgen, and check in the generated code. This introduces an issue for portability, as the code generation depends on the host we run bindgen. It is not a problem for now, as we only develop and test on one platform x86_64 linux. When we start considering about portability for the binding, we should run bindgen using Rust build script (build.rs).

qinsoon commented 1 year ago

Currently no Julia build flag is passed to bindgen. So bindgen treats all the conditional compilation flags as if they are not defined (unless defined on the system). We also removed compilation flags to make mmtk_julia_types.h work for our build environment, which means our current mmtk_julia_types.h is not portable.