-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
Remove the wasm32-wasi
target from rustc
#132562
base: master
Are you sure you want to change the base?
Conversation
This commit is the final step in the journey of renaming the historical `wasm32-wasi` target in the Rust compiler to `wasm32-wasip1`. Various steps in this journey so far have been: * 2023-04-03: rust-lang/compiler-team#607 - initial proposal for this rename * 2024-11-27: rust-lang/compiler-team#695 - amended schedule/procedure for rename * 2024-01-29: rust-lang#120468 - initial introduction of `wasm32-wasip1` * 2024-06-18: rust-lang#126662 - warn on usage of `wasm32-wasi` * 2024-11-08: this PR - remove the `wasm32-wasi` target The full transition schedule is in [this comment][comment] and is summarized with: * 2024-05-02: Rust 1.78 released with `wasm32-wasip1` target * 2024-09-05: Rust 1.81 released warning on usage of `wasm32-wasi` * 2025-01-09: Rust 1.84 to be released without the `wasm32-wasi` target This means that support on stable for the replacement target of `wasm32-wasip1` has currently been available for 6 months. Users have already seen warnings on stable for 2 months about usage of `wasm32-wasi` and stable users have another 2 months of warnings before the target is removed from stable. This commit is intended to be the final step in this transition so the source tree should no longer mention `wasm32-wasi` except in historical reference to the older name of the `wasm32-wasip1` target. [comment]: rust-lang#120468 (comment)
Some changes occurred in src/tools/compiletest cc @jieyouxu These commits modify compiler targets. Some changes occurred in src/doc/rustc/src/platform-support cc @Noratrieb |
@@ -377,7 +377,6 @@ target | std | host | notes | |||
`thumbv7a-pc-windows-msvc` | ✓ | | | |||
`thumbv7a-uwp-windows-msvc` | ✓ | | | |||
`thumbv7neon-unknown-linux-musleabihf` | ? | | Thumb2-mode Armv7-A Linux with NEON, musl 1.2.3 | |||
[`wasm32-wasip2`](platform-support/wasm32-wasip2.md) | ✓ | | WebAssembly |
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.
This is a bit unrelated to this PR, but I figured while I was here that I could go ahead and update the wasm32-wasip2
target from tier 3 to tier 2 to accurately reflect its status.
r? jieyouxu |
This also changes the LLVM/LLD target of wasm32-wasip1 to wasm32-wasip1. Is that intended to be done in this PR? |
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.
This looks mostly fine to me, r=me if the answer to Nora's question of changing the llvm target string is "this is intentional".
Ah yes sorry I forgot to call that out but it is indeed intentional. AFAIK it will have no practical change and it matches how @bors: r=jieyouxu |
This commit is the final step in the journey of renaming the historical
wasm32-wasi
target in the Rust compiler towasm32-wasip1
. Various steps in this journey so far have been:wasm32-wasi
target towasm32-wasi-preview1
compiler-team#607 - initial proposal for this renamewasm32-wasi
compiler-team#695 - amended schedule/procedure for renamewasm32-wasip1
target to rustc #120468 - initial introduction ofwasm32-wasip1
wasm32-wasi
#126662 - warn on usage ofwasm32-wasi
wasm32-wasi
targetThe full transition schedule is in this comment and is summarized with:
wasm32-wasip1
targetwasm32-wasi
wasm32-wasi
targetThis means that support on stable for the replacement target of
wasm32-wasip1
has currently been available for 6 months. Users have already seen warnings on stable for 2 months about usage ofwasm32-wasi
and stable users have another 2 months of warnings before the target is removed from stable.This commit is intended to be the final step in this transition so the source tree should no longer mention
wasm32-wasi
except in historical reference to the older name of thewasm32-wasip1
target.