search

JuliaHubOSS / llvm-cbe

resurrected LLVM "C Backend", with improvements
Other
826 stars 141 forks source link

Update README claims of LLVM support #122

Closed hikari-no-yume closed 3 years ago

hikari-no-yume commented 3 years ago

Re: https://github.com/JuliaComputingOSS/llvm-cbe/issues/119 and https://github.com/JuliaComputingOSS/llvm-cbe/pull/91.

Relevant to https://github.com/JuliaComputingOSS/llvm-cbe/issues/60, but does not implement it. As a practical matter: I am using LLVM 10 right now, there probably isn't full LLVM 11 support, and the latest version of LLVM is 12 which doesn't work yet (https://github.com/JuliaComputingOSS/llvm-cbe/issues/116).

Btw, I'm also wondering if we should remove the ifdef's for 9 and below.

hikari-no-yume commented 3 years ago

Do you have any opinion on what to do about the ifdefs?

vtjnash commented 3 years ago

I've typically stayed with a version until someone makes a PR to update, with the intent of only supporting one LLVM version, with that version being the newest that someone was able to get building and working.

vtjnash commented 3 years ago

But, per the linked issue, I haven't made that official, and so far have let ifdef's exist for a couple versions.

hikari-no-yume commented 3 years ago

Thanks for the response. I guess there's no need to close that door right now. Maybe if a future LLVM version makes things painful.