rustc_target: introduce Vendor, Abi, Env, Os #148531

tamird · 2025-11-05T15:04:28Z

Improve type safety by using an enum rather than strings.

I'm not really sure this is better since only a few vendors have special semantics. r? @nnethercote

rustbot · 2025-11-05T15:04:31Z

Some changes occurred in compiler/rustc_codegen_cranelift

cc @bjorn3

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

Some changes occurred in compiler/rustc_codegen_ssa

cc @WaffleLapkin

Some changes occurred in cfg and check-cfg configuration

cc @Urgau

The Miri subtree was changed

cc @rust-lang/miri

RalfJung · 2025-11-05T15:26:41Z

FWIW for a related change recently we had quite a bit of discussion: rust-lang/compiler-team#926.

nnethercote

I have a bunch of nits but this generally looks good. Linker is another candidate for enum-ification, if you aren't yet bored of this stuff :)

View changes since this review

compiler/rustc_target/src/spec/mod.rs

nnethercote · 2025-11-06T22:10:03Z

compiler/rustc_target/src/spec/mod.rs

+        Unikraft = "unikraft",
+        Uwp = "uwp",
+        Vex = "vex",
+        Win7 = "win7",


Any idea what "win7" is? Makes me think of Windows 7, which is a weird name for a vendor.

Dates back to #118150 which explains:

I went with naming the target x86_64-win7-windows-msvc, inserting the win7 in the vendor field (usually set to to pc). This is done to avoid ecosystem churn, as quite a few crates have cfg(target_os = "windows") or cfg(target_env = "msvc"), but nearly no cfg(target_vendor = "pc"). Since my goal is to be able to seamlessly swap to the win7 target, I figured it'd be easier this way.

Ugh, ok. Thanks for the info!

compiler/rustc_target/src/spec/mod.rs

compiler/rustc_codegen_cranelift/src/lib.rs

compiler/rustc_target/src/spec/base/apple/mod.rs

compiler/rustc_target/src/spec/mod.rs

compiler/rustc_codegen_cranelift/src/lib.rs

compiler/rustc_target/src/spec/mod.rs

nnethercote · 2025-11-07T02:03:56Z

FWIW for a related change recently we had quite a bit of discussion: rust-lang/compiler-team#926.

IMO these changes are so similar to the change in that MCP that additional discussion isn't needed.

tamird

Addressed all but one comment and left things unsquashed to (somewhat) ease review. I do plan to squash once you're happy with it.

View changes since this review

compiler/rustc_codegen_cranelift/src/lib.rs

compiler/rustc_target/src/spec/base/apple/mod.rs

compiler/rustc_target/src/spec/mod.rs

compiler/rustc_codegen_cranelift/src/lib.rs

tamird · 2025-11-07T03:47:40Z

compiler/rustc_target/src/spec/mod.rs

+        Unikraft = "unikraft",
+        Uwp = "uwp",
+        Vex = "vex",
+        Win7 = "win7",


Dates back to #118150 which explains:

I went with naming the target x86_64-win7-windows-msvc, inserting the win7 in the vendor field (usually set to to pc). This is done to avoid ecosystem churn, as quite a few crates have cfg(target_os = "windows") or cfg(target_env = "msvc"), but nearly no cfg(target_vendor = "pc"). Since my goal is to be able to seamlessly swap to the win7 target, I figured it'd be easier this way.

compiler/rustc_target/src/spec/mod.rs

compiler/rustc_target/src/lib.rs

nnethercote · 2025-11-07T06:05:27Z

r=me with the commits squashed, thanks!

@bors delegate=tamird

bors · 2025-11-07T06:05:30Z

✌️ @tamird, you can now approve this pull request!

If @nnethercote told you to "r=me" after making some further change, please make that change, then do @bors r=@nnethercote

tamird · 2025-11-07T12:07:56Z

range-diff is ~empty, which is what I expected!

@bors r=@nnethercote

bors · 2025-11-07T12:07:59Z

📌 Commit cd47df7 has been approved by nnethercote

It is now in the queue for this repository.

rustc_target: introduce Vendor, Abi, Env, Os Improve type safety by using an enum rather than strings. I'm not really sure this is better since only a few vendors have special semantics. r? `@nnethercote`

rustc_target: introduce Vendor, Abi, Env, Os Improve type safety by using an enum rather than strings. I'm not really sure this is better since only a few vendors have special semantics. r? ``@nnethercote``

bors · 2025-11-08T17:03:23Z

☔ The latest upstream changes (presumably #147935) made this pull request unmergeable. Please resolve the merge conflicts.

Avoid duplicating this comment in preparation for adding more enums.

Prepare for additional enums like Vendor and Os which have true `Unknown` variants. We want to use the same name for the escape hatch for all of these, thus rename this one.

Improve type safety by using an enum rather than strings.

rustbot · 2025-11-08T17:35:31Z

This PR was rebased onto a different master 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.

tamird · 2025-11-08T19:10:15Z

@bors r=@nnethercote

bors · 2025-11-08T19:10:17Z

📌 Commit 8f6aaa2 has been approved by nnethercote

It is now in the queue for this repository.

bors · 2025-11-08T21:24:18Z

☔ The latest upstream changes (presumably #148691) made this pull request unmergeable. Please resolve the merge conflicts.

rustbot added O-apple Operating system: Apple (macOS, iOS, tvOS, visionOS, watchOS) S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Nov 5, 2025

rustbot assigned nnethercote Nov 5, 2025

tamird force-pushed the vendor-enum branch 2 times, most recently from e47f7c3 to 013077b Compare November 6, 2025 03:08

rustbot added A-attributes Area: Attributes (`#[…]`, `#![…]`) A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. labels Nov 6, 2025

This comment has been minimized.

Sign in to view

tamird changed the title ~~rustc_target: introduce Vendor~~ rustc_target: introduce Vendor, Abi, Env, Os Nov 6, 2025

tamird changed the title ~~rustc_target: introduce Vendor, Abi, Env, Os~~ rustc_target: introduce Vendor, ABI, Env, OS Nov 6, 2025

tamird force-pushed the vendor-enum branch from 013077b to 0e0c808 Compare November 6, 2025 03:39

This comment has been minimized.

Sign in to view

tamird force-pushed the vendor-enum branch from 0e0c808 to dbd3aa2 Compare November 6, 2025 08:14

nnethercote requested changes Nov 7, 2025

View reviewed changes

rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 7, 2025

tamird force-pushed the vendor-enum branch from dbd3aa2 to 367f96e Compare November 7, 2025 04:03

tamird commented Nov 7, 2025

View reviewed changes

tamird changed the title ~~rustc_target: introduce Vendor, ABI, Env, OS~~ rustc_target: introduce Vendor, Abi, Env, Os Nov 7, 2025

nnethercote reviewed Nov 7, 2025

View reviewed changes

compiler/rustc_target/src/lib.rs Outdated Show resolved Hide resolved

tamird force-pushed the vendor-enum branch from 1bf1227 to cd47df7 Compare November 7, 2025 12:06

This comment has been minimized.

Sign in to view

bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Nov 7, 2025

matthiaskrgr mentioned this pull request Nov 8, 2025

Rollup of 10 pull requests #148693

Closed

bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Nov 8, 2025

tamird added 8 commits November 8, 2025 12:33

rustc_target: move comment to macro

6c5d3b2

Avoid duplicating this comment in preparation for adding more enums.

rustc_target: rename Arch::{Uknown,Other}

171875d

Prepare for additional enums like Vendor and Os which have true `Unknown` variants. We want to use the same name for the escape hatch for all of these, thus rename this one.

rustc_target: introduce Vendor

1cc730e

Improve type safety by using an enum rather than strings.

rustc_target: introduce Abi

536ea29

Improve type safety by using an enum rather than strings.

rustc_target: introduce Env

251344c

Improve type safety by using an enum rather than strings.

rustc_target: introduce Os

94eaaf2

Improve type safety by using an enum rather than strings.

rustc_target: rehome target_abi comment

d221d41

rustc_codegen_ssa: reduce repetition

8f6aaa2

tamird force-pushed the vendor-enum branch from cd47df7 to 8f6aaa2 Compare November 8, 2025 17:35

bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Nov 8, 2025

bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Nov 8, 2025

rustc_target: introduce Vendor, Abi, Env, Os #148531

Are you sure you want to change the base?

rustc_target: introduce Vendor, Abi, Env, Os #148531

Conversation

tamird commented Nov 5, 2025

Uh oh!

rustbot commented Nov 5, 2025

Uh oh!

RalfJung commented Nov 5, 2025

Uh oh!

This comment has been minimized.

This comment has been minimized.

nnethercote left a comment • edited by rustbot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

nnethercote Nov 6, 2025

Choose a reason for hiding this comment

Uh oh!

tamird Nov 7, 2025

Choose a reason for hiding this comment

Uh oh!

nnethercote Nov 7, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

nnethercote commented Nov 7, 2025

Uh oh!

tamird left a comment • edited by rustbot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

tamird Nov 7, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

nnethercote commented Nov 7, 2025

Uh oh!

bors commented Nov 7, 2025

Uh oh!

This comment has been minimized.

tamird commented Nov 7, 2025

Uh oh!

bors commented Nov 7, 2025

Uh oh!

bors commented Nov 8, 2025

Uh oh!

rustbot commented Nov 8, 2025

Uh oh!

tamird commented Nov 8, 2025

Uh oh!

bors commented Nov 8, 2025

Uh oh!

bors commented Nov 8, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

nnethercote left a comment •

edited by rustbot

Loading

tamird left a comment •

edited by rustbot

Loading