Description

Add llvm17-0 feature with LLVM 17 support.

This includes the following LLVM-C API changes:

Related Issue

No additional tests apart from the existing ones.

Disabled PassManagerBuilder for LLVM 17, as it was removed
Disabled the most of PassManager methods, as the corresponding functions were removed.

Pat-Lafon commented 1 year ago

Update the README 🙏 ?

SUPERCILEX commented 10 months ago

Any updates on this?

TheDan64 commented 10 months ago

@vadorovsky We can merge this once the test suite is fixed to work with other versions

novafacing commented 10 months ago

Is anything needed other than to add entries for 17-0 to test.yml to enable CI? I can help out if needed.

SUPERCILEX commented 10 months ago

I put out a PR that gets us over the edge. Was just an import typo: https://github.com/TheDan64/inkwell/pull/460

SUPERCILEX commented 10 months ago

Ah shoot, spoke too soon. All the tests and examples are broken on llvm 17.

vadorovsky commented 10 months ago

Whoops, sorry for lack of reaction! I will try to fix the tests later today. I will also add README changes.

SUPERCILEX commented 10 months ago

That'd be great! You can grab my commit here: f5a1771 (#460)