rpav / c2ffi

Clang-based FFI wrapper generator
GNU Lesser General Public License v2.1
234 stars 39 forks source link

Updates for llvm-15 and llvm-16? #114

Closed hjudt closed 1 year ago

hjudt commented 1 year ago

Any chance of updates (aka branches/releases) for llvm-15 and llvm-16?

There are build fixes for llvm-15, would it be possible to create a branch with this fix? I am a package maintainer and keeping up this organizational ado would make it less cumbersome having to deal with the updates.

rpav commented 1 year ago

Right! I'll try and get this out this weekend.

rpav commented 1 year ago

Like two weeks later but I have actually not forgotten about this, just not quite gotten there.

rpav commented 1 year ago

I finally got everything in order and have started work for clang-15; in terms of building, it mostly looks like they changed init options around a bit. It's hard to tell yet if this is the only major change.

hjudt commented 1 year ago

Ok, thanks a lot for working on this!

hjudt commented 1 year ago

Hi! Any updates on this?

rpav commented 1 year ago

Yeah finally have a preliminary and apparently-working llvm-15 version.

Ditched NixOS for versioning and am currently working on an alpine-based container build that will hopefully be an "offically-supported" way to build and run c2ffi with arbitrary versions of llvm/clang, but need to minimize some user complexity. Hopefully once this works clang-16 should be next.

hjudt commented 1 year ago

Thank you very much for your work on llvm-15, it worked and compiled fine and I was able to provide an updated version in the gentoo portage tree.

Are you still working on an updated llvm-16 version? In gentoo, version 16 has already been stabilized which means older versions will be deprecated and removed in a few weeks/months, so it would be nice to have updates rather sooner than later...

hjudt commented 1 year ago

hi! any updates for llvm-16?

rpav commented 1 year ago

Life has been dumb but hopefully soon.

hjudt commented 1 year ago

Any news? llvm-17 has already arrived...

rpav commented 1 year ago

This is updated for llvm-16 .. it didn't even require changes other than bumping the required version. :P It would be helpful for people using it to try / post an error log.

llvm-17 doesn't appear to be officially here yet; I'll probably get to it when Arch or Alpine post packages. If you're working on prerelease, also test to see there are even any changes required; there may not be once again.

hjudt commented 1 year ago

Thank you. llvm-16 builds fine here, so I can issue a pull request for it now in the gentoo repo.

llvm-17 is still unstable in the portage package repository, so that might still take some while to stabilize, but simply bumping the version to 17 gives me a compile error for AST.cpp, I'll open another issue for it and append the output there.