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
llvmPackages_latest: llvm9->llvm10 #84096
Conversation
pkgs/top-level/all-packages.nix
Outdated
llvm-polly = llvmPackages_latest.llvm-polly; | ||
clang-polly = llvmPackages_latest.clang.override { cc = llvmPackages_latest.clang-polly-unwrapped; }; | ||
|
||
# llvm and clang are build with polly by default |
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.
# llvm and clang are build with polly by default | |
# llvm and clang are built with polly by default |
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.
After looking how this is usually done, I changed the lines to use throw. This prevents llvm-polly
/clang-polly
, but also provides an error message to the user. I think this is the best solution.
Thanks for finding the typo :)
I guess this should be |
This causes almost no rebuilds, because |
since llvm10 is now the newest release now throws an error if llvm-polly or clang-polly is build, since polly is enabled by default.
b8a6b81
to
c4afa32
Compare
pkgs/top-level/all-packages.nix
Outdated
clang-polly = llvmPackages_latest.clang.override { cc = llvmPackages_latest.clang-polly-unwrapped; }; | ||
|
||
llvm-polly = throw "clang is now built with polly-plugin by default"; | ||
clang-polly = throw "clang is now built with polly-plugin by default"; |
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.
Put this in pkgs/top-level/aliases.nix
with a timestamp.
Motivation for this change
Since llvm10 is now the newest release.
This also sets llvm-polly and clang-polly to normal llvm/clang, since polly is now enabled by default.
Maybe we want to remove the
-polly
variants at some point in the future?The only affected packages are:
Things done
The tiledb python module has tests failing, but this is unrelated to llvm (see #84091 for more information).
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)