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
llvm: Remove unneeded libcxxabi dependencies #41952
llvm: Remove unneeded libcxxabi dependencies #41952
Conversation
Success on x86_64-linux (full log) Attempted: llvm Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: llvm Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: llvm Partial log (click to expand)
|
Another thing i forgot about is putting cmake as native build input everywhere. |
I got these builds in progress; That one I'm happy to just put through ofborg as it's basically a no-op for native. |
Finish what 1cb1b91 started.
deb8f43
to
88c9cf7
Compare
Evidentally this hasn't been needed for a while
88c9cf7
to
7156f46
Compare
Testing showed that every LLVM since 3.7 worked fine (though 6 was untested). Guess this hasn't been needed for a while! |
Trivial changes vs last ofborg eval, and tested by hand. Merging. |
Success on aarch64-linux (full log) Attempted: llvm Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: llvm Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: llvm Partial log (click to expand)
|
@@ -43,7 +43,8 @@ in stdenv.mkDerivation (rec { | |||
++ stdenv.lib.optional enableManpages python.pkgs.sphinx; | |||
|
|||
buildInputs = [ libxml2 libffi ] | |||
++ stdenv.lib.optionals stdenv.isDarwin [ libcxxabi ]; | |||
# TODO(@Ericson2314): Remove next mass rebuild | |||
++ stdenv.lib.optionals (stdenv.isDarwin && stdenv.hostPlatform == stdenv.buildPlatform) [ libcxxabi ]; |
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.
Did you test an stdenv build with this? Don't want us to forget about it and run into surprising issues with llvm 5 -> 6 later on.
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.
@LnL7 I just built llvm 6 didn't use a it bunch. I am working on this for LLVM 5 too with bootstrapping, so I guess it's retroactively tested? :/
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.
For posterity, indeed staging now has the same changes with #42048
Motivation for this change
I'm pretty sure this is just there for bootstrapping Darwin for some reason; normal builds of llvm given a regular stdenv don't need it for Darwin and Linux alike. Testing now to confirm.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)