Support llvm-15 for inkwell

[ksolana]

Description

Added llvm-15 support
#359

Issues

Still has build errors.

argo build
   Compiling inkwell v0.1.0 (/run/media/inkwell)
error: One of the LLVM feature flags must be provided: llvm4-0 llvm5-0 llvm6-0 llvm7-0 llvm8-0 llvm9-0 llvm10-0 llvm11-0 llvm12-0 llvm13-0 llvm14-0 llvm15-0 
   --> src/lib.rs:97:9
    |
97  |         compile_error!(concat!("One of the LLVM feature flags must be provided: ", $($all, " "),*));
    |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
...
108 | assert_unique_used_features! {"llvm4-0", "llvm5-0", "llvm6-0", "llvm7-0", "llvm8-0", "llvm9-0", "llvm10-0", "llvm11-0", "llvm12-0", "llvm13-0", "llvm14-0", "llvm15-0"}
    | ----------------------------------------------------------------------------------------------------------------------------------------------------------------------- in this macro invocation
    |
    = note: this error originates in the macro `assert_used_features` which comes from the expansion of the macro `assert_unique_used_features` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0433]: failed to resolve: use of undeclared crate or module `llvm_sys`
 --> src/support/error_handling.rs:4:5
  |
4 | use llvm_sys::core::{LLVMGetDiagInfoDescription, LLVMGetDiagInfoSeverity};
  |     ^^^^^^^^ use of undeclared crate or module `llvm_sys`

[TheDan64]

Still has build errors.

As mentioned in the issue, it's not just as simple as updating the Cargo.toml :)

Off the top of my head, there's at least a few more things needed:

• https://github.com/TheDan64/inkwell/blob/master/internal_macros/src/lib.rs#L17
• https://github.com/TheDan64/inkwell/blob/master/src/values/metadata_value.rs#L36
• Add a new llvm15 build to .github/workflow/test.yml
• Fix any failing tests

And probably more..

[ksolana]

This macro still errors out. What is it looking for?

$ cargo build
   Compiling inkwell v0.1.0 (/run/media/adityak/tb_half/inkwell)
error: One of the LLVM feature flags must be provided: llvm4-0 llvm5-0 llvm6-0 llvm7-0 llvm8-0 llvm9-0 llvm10-0 llvm11-0 llvm12-0 llvm13-0 llvm14-0 llvm15-0 
   --> src/lib.rs:97:9
    |
97  |         compile_error!(concat!("One of the LLVM feature flags must be provided: ", $($all, " "),*));
    |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
...
108 | assert_unique_used_features! {"llvm4-0", "llvm5-0", "llvm6-0", "llvm7-0", "llvm8-0", "llvm9-0", "llvm10-0", "llvm11-0", "llvm12-0", "llvm13-0", "llvm14-0", "llvm15-0"}
    | ----------------------------------------------------------------------------------------------------------------------------------------------------------------------- in this macro invocation

[TheDan64]

Need to specify an llvm version feature flag

[ksolana]

Need to specify an llvm version feature flag

Can you point to the code where I need to specify that?

[TheDan64]

cargo build --features llvm15-0

[gavrilikhin-d]

...And add llvm15-0 as supported version to README 😀

[gavrilikhin-d]

But there is a problem. LLVM is moving to opaque pointers. In llvm 15 non-opaque pointer access may already have been removed from API

[TheDan64]

Opaque ptrs are still optional in LLVM15, we can figure out come the LLVM16 release

[ksolana]

Fixed cargo build --features llvm15-0. Still some work is required to fix cargo test --features llvm15-0

[hedgar2017]

@aditya-solana I've finished support in this PR. Please check if you'd like to add anything.
But I'd like to run the CI before merging. Unfortunately, it doesn't work for LLVM 15.

[ksolana]

@hedgar2017
Thanks for your efforts!

[hedgar2017]

@aditya-solana I think you may close this PR. We'll wait until the CI is fixed and then merge mine.

[ksolana]

closing