TheDan64 / inkwell

It's a New Kind of Wrapper for Exposing LLVM (Safely)

Home Page:https://thedan64.github.io/inkwell/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Support llvm 17

Qwinci opened this issue · comments

commented

Add support for llvm 17, it removes the legacy optimization pipeline so this might be a decent amount of work to do, though I am not completely if all the PassManager stuff except the PassBuilderOptions and Module.run_passes() should just be disabled in llvm versions >= 17.

There is a bug in MCJIT in llvm 16 I've experienced today, that was fixed in newer version.

So I was wondering, when inkwell will support llvm 17?

I think, you may just provide a feature for every version that llvm-sys supports and remove/deprecate unsupported methods. Then you could add new API later