search

TheDan64 / inkwell

It's a New Kind of Wrapper for Exposing LLVM (Safely)
https://thedan64.github.io/inkwell/
Apache License 2.0
2.32k stars 224 forks source link

Support llvm 17 #444

Closed Qwinci closed 8 months ago

Qwinci commented 1 year ago

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.

gavrilikhin-d commented 11 months ago

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