Skip to content

Commit

Permalink
llvmPackages_10: removed extra polly-build
Browse files Browse the repository at this point in the history
There is no good reason to have separate builds for polly and no-polly
versions. wwwwwThe reason for the two versions was (as far as I can
tell) to defer rebuilds (see ed60483).
Polly is now enabled by default.
  • Loading branch information
DieGoldeneEnte committed Mar 19, 2020
1 parent 0ec3f4e commit e9aa877
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 6 deletions.
5 changes: 0 additions & 5 deletions pkgs/development/compilers/llvm/10/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -30,16 +30,11 @@ let
in {

llvm = callPackage ./llvm.nix { };
llvm-polly = callPackage ./llvm.nix { enablePolly = true; };

clang-unwrapped = callPackage ./clang {
inherit (tools) lld;
inherit clang-tools-extra_src;
};
clang-polly-unwrapped = callPackage ./clang {
inherit clang-tools-extra_src;
llvm = tools.llvm-polly;
};

llvm-manpages = lowPrio (tools.llvm.override {
enableManpages = true;
Expand Down
2 changes: 1 addition & 1 deletion pkgs/development/compilers/llvm/10/llvm.nix
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
, enablePFM ? !(stdenv.isDarwin
|| stdenv.isAarch64 # broken for Ampere eMAG 8180 (c2.large.arm on Packet) #56245
)
, enablePolly ? false
, enablePolly ? true
}:

let
Expand Down

0 comments on commit e9aa877

Please sign in to comment.