-
Notifications
You must be signed in to change notification settings - Fork 108
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
Calling Dex code from Julia via sharing LLVM bitcode #634
Comments
We could start with adding dex to Yggdrasil, building against our specific LLVM there. |
Julia needs a patched LLVM. Are those patches compatible with Dex? Does it have its own patches? |
I think Julia's LLVM is not very patched? I would expect that they still can talk same bytecode? Dex does not use any patches on llvm afaik. |
We should still be bitcode compatible with an unpatched version, as long as the optimizations are run by the Julia LLVM, (and we are upstreaming our patch list) |
Yeah we don't need any LLVM changes. If the Julia patches are minor then I don't expect any significant difficulties. Right now we can support LLVM 9 or LLVM HEAD, but it shouldn't be hard to add LLVM 12 support as well. |
From #625 (review)
I asked @vchuravy about how realistic that was and they said:
Apparently, this is not a reason Cxx.jl is incredibly fragile.
So this seems possible.
Normal builds of Julia 1.6 is LLVM v11, Julia 1.7 is LLVM v12.
I think there is a long of swing possible when compiling from source but its pretty rare for people to do that and so don't want to depend on it.
The text was updated successfully, but these errors were encountered: