-
Notifications
You must be signed in to change notification settings - Fork 126
Description
Since anyone last touched our FFI layer, the LLVM C API has been updated significantly. Most importantly, the LLVM C API now has functions which do things for which we have hand-rolled code in our FFI.
Where the LLVM C API now has a function that does something, and we have hand-rolled code that does something, we should remove our hand-rolled code and instead call the LLVM C API function.
This is quite a large undertaking, effectively being an audit of the entire FFI layer. However, it can be done piecewise, module-by-module.
This will result in our FFI layer becoming quite a lot smaller and simpler, which will make it easier to maintain going forward. It also reduces the likelihood of something breaking dues to a change in upstream LLVM, since we can assume that the LLVM project will ensure that their own C API is working.