-
Notifications
You must be signed in to change notification settings - Fork 572
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add recipe for the LLVM SPIR-V back-end. #10516
Conversation
789799a
to
d4643bd
Compare
You want to delete https://github.com/JuliaBinaryWrappers/SPIRV_LLVM_Backend_jll.jl/releases/tag/SPIRV_LLVM_Backend-v0.1.0%2B0 otherwise registration will fail |
using BinaryBuilder, Pkg | ||
|
||
name = "SPIRV_LLVM_Backend" | ||
version = v"19" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only loosely following LLVM's versioning here so that we can still make minor releases ourselves (e.g. containing Julia-specific patches).
Ah, good call. I accidentally deployed that by doing I've just deleted the entire repo. |
Turns out the back-end from LLVM 19 is pretty unusable, so I'll bump this to 20 straight away (sorry for wasting some compute time here). |
LLVM 20 failing to build? |
Yggdrasil/S/SDL2/build_tarballs.jl Lines 25 to 29 in 52fe346
|
For LLVM specifically, others have simply patched out the offending file: Yggdrasil/C/Cling/bundled/patches/avoid_builtin_available.patch Lines 1 to 13 in 52fe346
https://github.com/llvm/llvm-project/blob/ff8f6abe20a9dfbb1de8d978f865092413d3416c/llvm/lib/Support/Unix/Path.inc#L1490 |
To check if it's possible to move from https://github.com/KhronosGroup/SPIRV-LLVM-Translator to LLVM proper.
Doesn't need platform augmentations, because I'd like to simply use the latest version of the back-end possible. This works as long as we statically link
libLLVM
, but may still pose problems when there's IR incompatibilities that can't get auto-upgraded.cc @vchuravy