search

AdeptLanguage / Adept

The Adept Programming Language
GNU General Public License v3.0
119 stars 9 forks source link

Compatibility with LLVM 17 #191

Closed ghost closed 11 months ago

ghost commented 11 months ago

Adept/src/BKEND/ir_to_llvm.c:5:10: fatal error: 'llvm-c/Transforms/IPO.h' file not found

It seems LLVM 17 removed that file:

$ ls /clang64/include/llvm-c/Transforms
PassBuilder.h
$ clang -v
clang version 17.0.1
Target: x86_64-w64-windows-gnu
Thread model: posix
InstalledDir: C:/msys64/clang64/bin
IsaacShelton commented 11 months ago

Yes LLVM support needs to be upgraded. They always like making breaking changes so it'll be fun to upgrade.

I'll take a look at it

IsaacShelton commented 11 months ago

LLVM 17 is not released on CI/CD runners yet, but it does work if you choose to use it.

All releases have been updated to use LLVM 16 for the time being.

ghost commented 11 months ago

All releases have been updated to use LLVM 16 for the time being.

Is compatibility with older releases of LLVM also removed? Even though I think Adept should work with the latest LLVM, I don't think the mindset of requiring latest/greatest version is right.

IsaacShelton commented 11 months ago

Yes it still works with older versions