-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
Make llvm.download-ci-llvm="if-available"
work for tier 2 targets with host tools
#89888
Make llvm.download-ci-llvm="if-available"
work for tier 2 targets with host tools
#89888
Conversation
(rust-highfive has picked a reviewer for you, use r? to override) |
rust.download-ci-llvm="if-available"
work for aarch64-apple-darwinllvm.download-ci-llvm="if-available"
work for aarch64-apple-darwin
Hm. In theory, the list of targets is probably the tier 2 with host tools -- roughly (or "better"). We can probably add all of those listed there -- https://doc.rust-lang.org/nightly/rustc/platform-support.html#tier-2-with-host-tools I don't think we need automate the if-available set but that seems like a better delta here probably? |
llvm.download-ci-llvm="if-available"
work for aarch64-apple-darwinllvm.download-ci-llvm="if-available"
work for tier 2 platforms with host tools
16d1aec
to
ac298c9
Compare
@rustbot label -S-waiting-on-author +S-waiting-on-review |
PR description edited and force-pushed to include all tier 2 targets with host tools. |
llvm.download-ci-llvm="if-available"
work for tier 2 platforms with host toolsllvm.download-ci-llvm="if-available"
work for tier 2 targets with host tools
@bors r+ rollup Thanks! |
📌 Commit ac298c9 has been approved by |
…askrgr Rollup of 7 pull requests Successful merges: - rust-lang#89738 (ty::pretty: prevent infinite recursion for `extern crate` paths.) - rust-lang#89888 (Make `llvm.download-ci-llvm="if-available"` work for tier 2 targets with host tools) - rust-lang#89945 (Remove a mention to `copy_from_slice` from `clone_from_slice` doc) - rust-lang#89946 (Fix an ICE with TAITs and Future) - rust-lang#89963 (Some "parenthesis" and "parentheses" fixes) - rust-lang#89975 (Add a regression test for rust-lang#85921) - rust-lang#89977 (Make Result::as_mut const) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
llvm.download-ci-llvm="if-available"
is used for most profiles configured viax.py setup
. It allows downloading prebuilt LLVM tarballs from the CI artifacts for a configured list of platforms. Currently this list is restricted to tier 1 targets but it makes sense for all tier 2 targets with host tools.