Skip to content

RFC: Promote aarch64-pc-windows-msvc to Tier 1 #3817

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dpaoliello
Copy link
Contributor

@dpaoliello dpaoliello commented May 22, 2025

@dpaoliello dpaoliello force-pushed the aarch64tier1 branch 2 times, most recently from 9594548 to 359127e Compare May 22, 2025 21:56
Comment on lines 116 to 119
[ci-draft-pr][https://github.com/rust-lang/rust/pull/140136]
[llvm-builders][https://lab.llvm.org/buildbot/#/builders/161]
[platform-support][https://github.com/rust-lang/rust/blob/e3892a40a9d06034fdf2432a9d3d29fa97726299/src/doc/rustc/src/platform-support.md?plain=1#:~:text=aarch64%2Dpc%2Dwindows%2Dmsvc]
[promote-tier-2][https://github.com/rust-lang/rust/pull/75914]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The syntax for link definitions needs to have a colon and look like:

[link-name]: https://example.com/


> The target maintainer team must include at least 3 developers.

There is an existing [Windows team for Rust][windows-team] and, within Microsoft, Arm64 Windows is
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are there at least 3 people in that group that are willing to be active maintainers for this particular target? It seems like we should identify who those people will be.

The windows marker team is mainly just a ping group and not necessarily maintainers (anyone can be added to it).

Copy link
Member

@jieyouxu jieyouxu May 23, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Problem (blocking): Yeah, Tier 1 target policy requires

The target maintainer team must include at least 3 developers.

The "windows" ping group is advisory at best, it is not even specific to the other Tier 1 x86_64-pc-windows-msvc or x86_64-pc-windows-gnu targets1.

On the compiler team, we lack expertise bandwidth on the Tier 1 Windows targets already. Furthermore, to the best of my knowledge, very few (if any) compiler team members have access to a aarch64-pc-windows-msvc host, be it natively or remotely. This makes even "routine" tasks like triaging or bisection or minimizations very difficult, so for a Tier 1 target, we would have to rely on its target maintainers perhaps more than is the case for other Tier 1 targets.

Footnotes

  1. In fact, those Tier 1 targets don't technically adhere to our current Target Policy, because they were grandfathered in historically (it obviously made sense at the time, and still does now), but the compiler team would like to have actual dedicated target maintainers for the other Tier 1 Windows targets too going forward.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fair enough: let me talk to a few folks internally, I'd like to get an official *-pc-windows-msvc maintainer list setup for the various Windows targets.


> All requirements for tier 2 apply.

[`aarch64-pc-windows-msvc` is already a Tier 2 target.][platform-support]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AFAIK aarch64-pc-windows-msvc was grandfathered into the tier policy without having its requirements verified. I don't think there is anything particularly noteworthy, but I think it would probably be good to not gloss over them.

Copy link
Member

@RalfJung RalfJung May 23, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The aarch64 windows target is listed on rust-lang/rust#113739 because it does not currently have any target maintainers listed. So yes, there is something quite noteworthy here. We should really go over that entire checklist.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, expanded with the relevant points from Tier 2.

@ehuss ehuss added T-compiler Relevant to the compiler team, which will review and decide on the RFC. T-infra Relevant to the infrastructure team, which will review and decide on the RFC. T-release Relevant to the release team, which will review and decide on the RFC. labels May 23, 2025
@programmerjake
Copy link
Member

related: promoting *-gnullvm to tier 2 with host tools rust-lang/compiler-team#877

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-compiler Relevant to the compiler team, which will review and decide on the RFC. T-infra Relevant to the infrastructure team, which will review and decide on the RFC. T-release Relevant to the release team, which will review and decide on the RFC.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants