-
-
Notifications
You must be signed in to change notification settings - Fork 14.8k
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
Build failure: rocmPackages.llvm.libc #368672
Comments
Can confirm that |
Build succeeds:
Build fails:
Looks like #361878? Sorry, I'm not sure I have the resources to bisect which of the staging-next commits changed expectations here. |
Currently bisecting this. |
first bad commit: [a1c1180] libuv: 1.48.0 -> 1.49.2 It seems I took a way too early bad commit. As this error has nothing to do with the current issue. |
fyi: be aware that the merged staging next bumped gcc13 to gcc14. If i saw it correctly when trying to address it at #197885 (comment) the failure was then building the stage1 compiler (which is compiled by gcc). #367695 does an overall bump and with that the observed error was not existing for me. |
This reverts commit e05446c. See NixOS/nixpkgs#368672
Don't update nixpkgs until NixOS/nixpkgs#368672 is fixed
Of course, because of this, packages like |
maybe unrelated but amd-libflame fails - as well several cuda packages because of gcc14 (nvcc must use gcc13) |
It certainly seems possible. Hopefully it is trivial to try a revert and see if that fixes things? I will have little time to do this in the near future. /cc @emilazy who may be interested. |
It seems unlikely to me, but I guess it depends on if the ROCm build system is doing weird things with unwrapped Clang. Edit: Okay, it is using an unwrapped Clang. Try getting |
Evidence for this, note the compiler is
I was able to get NIX_DEBUG into the args and see that NIX_DEBUG output in the expanded details below.
There is probably something obvious wrong here but I don't have a lot of attention on this right now. |
There seems to be something fishy here:
I think nixpkgs/pkgs/build-support/cc-wrapper/default.nix Lines 596 to 599 in 4f0dadb
...but nixpkgs/pkgs/development/rocm-modules/6/llvm/stage-1/runtimes.nix Lines 1 to 32 in 4f0dadb
I don't understand enough to be sure, but Edit: Opened #370180. |
IMO the proper solution is that the ROCm LLVM should never itself be an We should just build a monolithic derivation of ROCm LLVM containing See #370435 for an example of how this would work. The core changes (which I would consider simplifications) are in |
Side note, to make the cc-wrapper logic of #370435 robust (i.e. to get rid of the |
FWIW I really think that ROCm’s LLVM should be part of the main |
FYI this seems to affect rocmPackages_5 aswell
|
@ninelore Can you try again on master? |
lgtm |
Seeing the same issue here as well #365718 |
What commit are you testing? Does rebasing on top of current master help? |
hasn't hit unstable yet https://nixpk.gs/pr-tracker.html?pr=370180 |
@collares I see it's hit nixos-unstable-small I'm testing against that right now and the build has gotten further than before. Will report on success/fail after I complete a few tests. |
Thanks! It might be best to file follow-up issues or to discuss failures in the ollama PR, depending on where the problem is. |
hey, it works! many thanks |
not sure if its related but encountered this on updating unstable and rebuilding:
amd gpu + rocm config:
|
This failure is for a different package, and probably a different failure. Please open a new ticket instead. |
…a again, because of kernel 6.13 Signed-off-by: Patrizio Bekerle <[email protected]>
For kernel 6.12 this issue was fixed for me, with kernel 6.13 I get the llvm errors again. 😅 |
yeah, I just had to switch back to stable - find it hard to be without |
Steps To Reproduce
Steps to reproduce the behavior:
nix build github:NixOS/nixpkgs#rocmPackages.llvm.libc
Build log
log.txt
Short version
Additional context
Metadata
"x86_64-linux"
Linux 6.6.66-hardened1, NixOS, 25.05 (Warbler), 25.05.20241219.d70bd19
yes
yes
nix-env (Nix) 2.24.11
/nix/store/8fwsiv0hd7nw1brkvka0jf1frk3m7qkr-source
Notify maintainers
@acowley
@lovesegfault
Note for maintainers: Please tag this issue in your PR.
Add a 👍 reaction to issues you find important.
The text was updated successfully, but these errors were encountered: