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_6, llvm_7: build all default targets #53941
Conversation
@@ -28,9 +26,6 @@ let | |||
shortVersion = with stdenv.lib; | |||
concatStringsSep "." (take 1 (splitString "." release_version)); | |||
|
|||
inherit | |||
(import ../common.nix { inherit (stdenv) lib; }) | |||
llvmBackendList; |
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.
Do we still need common.nix in that case?
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.
No, indeed. Removed.
Assuming it evals, 👍 :).
…On Mon, 14 Jan 2019 17:35:49 +0000 (UTC), Orivej Desh ***@***.***> wrote:
orivej commented on this pull request.
> @@ -28,9 +26,6 @@ let
shortVersion = with stdenv.lib;
concatStringsSep "." (take 1 (splitString "." release_version));
- inherit
- (import ../common.nix { inherit (stdenv) lib; })
- llvmBackendList;
No, indeed. Removed.
--
You are receiving this because your review was requested.
Reply to this email directly or view it on GitHub:
#53941 (comment) part: text/html
|
Did aarch64 build succeed before? |
At least llvm_7 used to build on aarch64, because we are using this version for rust. |
llvm_7 still builds on aarch64, but llvm_6 needs a backport of this fix: https://reviews.llvm.org/D50583. (This is similar to an older https://reviews.llvm.org/D28285.) |
This makes LLVM tools (including dependent tools such as LLD) readily useful in more situations, foresees such needed additions as BPF and NVPTX, and brings llvm_6 and newer on par with the current default llvm_5.
@GrahamcOfBorg build llvm_6 llvm_7 |
Motivation for this change
This makes LLVM tools (including dependent tools such as LLD) readily useful in
more situations, foresees such future additions as BPF and NVPTX, and brings
llvm_6 and newer on par with the current default llvm_5.
The list of targets was reduced in #52031, which was not necessary to fix cross compilation.
BPF target is needed for BCC in #53920.
out
andlib
size increases by 1/3, but the full LLVM is needed regardless, and this avoids having a partial LLVM.Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)