-
Notifications
You must be signed in to change notification settings - Fork 571
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
[Reactant] Build for aarch64-linux with CUDA #10313
[Reactant] Build for aarch64-linux with CUDA #10313
Conversation
9244126
to
97b95a7
Compare
e3d0b6a
to
633fa57
Compare
Spoiler: local build is successful, but we still need to do reorganise some bits.
af7b742
to
93dd806
Compare
https://buildkite.com/julialang/yggdrasil/builds/16785#0194a869-1348-441d-a6a6-80a72a8b2a2c/687-1725
Sigh. I've seen this error before, but can't remember where. Edit: that's where I saw it: #10190 (comment) |
Doing a quick search in https://github.com/roblabla/MacOSX-SDKs I think that means we need to use at least the 13.3 SDK: % grep -r 'hardware_destructive_interference_size' .
./MacOSX13.3.sdk/usr/include/c++/v1/new:inline constexpr size_t hardware_destructive_interference_size = __GCC_DESTRUCTIVE_SIZE;
./MacOSX14.0.sdk/usr/include/c++/v1/new:inline constexpr size_t hardware_destructive_interference_size = __GCC_DESTRUCTIVE_SIZE;
./MacOSX14.2.sdk/usr/include/c++/v1/new:inline constexpr size_t hardware_destructive_interference_size = __GCC_DESTRUCTIVE_SIZE;
./MacOSX14.4.sdk/usr/include/c++/v1/new:inline constexpr size_t hardware_destructive_interference_size = __GCC_DESTRUCTIVE_SIZE;
./MacOSX14.5.sdk/usr/include/c++/v1/new:inline constexpr size_t hardware_destructive_interference_size = __GCC_DESTRUCTIVE_SIZE;
./MacOSX15.0.sdk/usr/include/c++/v1/new:inline constexpr size_t hardware_destructive_interference_size = __GCC_DESTRUCTIVE_SIZE; I hope that's enough..... |
Small progress, with the 13.3 SDK I get to a different error:
Edit: git bisect points to openxla/xla@9d95a71. Edit: this was fixed by openxla/xla#21768, we were building on top of the previous commit 🥲 |
452d696
to
38aeca6
Compare
https://buildkite.com/julialang/yggdrasil/builds/16793#0194a97d-d9de-402f-99e3-30ce6338f7b7/6-20170
@wsmoses this is your stuff, it was working before EnzymeAD/Reactant.jl@ |
R/Reactant/build_tarballs.jl
Outdated
# Compiling LLVM components within XLA requires macOS SDK 10.14. | ||
pushd $WORKSPACE/srcdir/MacOSX10.*.sdk | ||
# But then we need 13.3 for another `std::hardware_destructive_interference_size`: | ||
# <https://github.com/JuliaPackaging/Yggdrasil/pull/10313#issuecomment-2616125018>. |
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.
@giordano we were just removing hw interference size: https://github.com/EnzymeAD/Reactant.jl/blob/fd60aad440379a9b04a0164f34cbd65cd679ed89/deps/ReactantExtra/WORKSPACE#L57
This reverts commit 909a78c.
76d6cb8
to
b189eda
Compare
🚀 |
1 similar comment
🚀 |
This is only to save my progress along the way, nothing works at the moment.