Promote riscv64-unknown-linux-musl to tier 2 with host tools

[eshattow]

View all comments

MCP: rust-lang/compiler-team#982

Implements #156191

[rustbot]

This PR changes how LLVM is built. Consider updating src/bootstrap/download-ci-llvm-stamp.

Warning

If you are changing how CI LLVM is built or linked, make sure to bump
src/bootstrap/download-ci-llvm-stamp.

cc @jieyouxu

These commits modify compiler targets.
(See the Target Tier Policy.)

Some changes occurred in src/doc/rustc/src/platform-support

cc @Noratrieb

[rustbot]

Thanks for the pull request, and welcome! The Rust Project is excited to review your changes, and you should hear from @marcoieni (or someone else) some time within the next two weeks.

Please see the contribution instructions for more information. Namely, in order to ensure the minimum review times lag, PR authors and assigned reviewers should ensure that the review label (S-waiting-on-review and S-waiting-on-author) stays updated, invoking these commands when appropriate:

• @rustbot author: the review is finished, PR author should check the comments and take action accordingly
• @rustbot review: the author is ready for a review, this PR will be queued again in the reviewer's queue

Why was this reviewer chosen?

The reviewer was selected based on:

• Owners of files modified in this PR: infra-ci
• infra-ci expanded to Kobzol, Mark-Simulacrum, jdno, jieyouxu, marcoieni
• Random selection from Mark-Simulacrum, jdno, jieyouxu, marcoieni

[marcoieni]

@bors try jobs=dist-riscv64-linux-musl,dist-riscv64-linux-gnu

[marcoieni]

Maybe the failure is due to this codex finding, which I copy-paste without editing:

---

[P1] New Docker image does not copy the crosstool-ng checksum patch.
Dockerfile:6 runs crosstool-ng.sh, but that script unconditionally reads /scripts/crosstool-ng-sha256-20260705.diff at crosstool-ng.sh:20. The existing RISC-V GNU image copies that file before running the script.

[rust-bors]

💔 Test for 9d14c25 failed: CI. Failed job:

• try - dist-riscv64-linux-musl (web logs, enhanced plaintext logs)

[eshattow]

@marcoieni writes:

Maybe the failure is due to this codex finding, which I copy-paste without editing:
...

I'd missed this, as it is a recent change to resolve the fallout of kernel.org mirrors CDN being down for several days.

Added the appropriate COPY line to the Dockerfile and pushed, thanks for the suggestion :-)

[rustbot]

This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

[eshattow]

@bors try jobs=dist-riscv64-linux-musl,dist-riscv64-linux-gnu

[rust-bors]

@eshattow: 🔑 Insufficient privileges: not in try users

[marcoieni]

@bors try jobs=dist-riscv64-linux-musl,dist-riscv64-linux-gnu

[rust-bors]

☀️ Try build successful (CI)
Build commit: af6bde0 (af6bde0aa65b20bb4a0f8294334d9ab4ceff9eaa)
Base parent: 31e4b84 (31e4b8481fec5ae856d786ff3b75a5bccaf33725)

[eshattow]

@marcoieni this build has been tested successfully on riscv64 hardware, there is some missing address sanitizer coverage of both -gnu and -musl targets that would be appropriate for a follow-up issue but is not any blocker here now. I want to know any obvious problems with documentation changes or omissions. Also is there any requirement to add myself as a code maintainer? Thanks for your assistance, -E