Skip to content

Classify Winch with Tier 1 support #10755

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: main
Choose a base branch
from

Conversation

alexcrichton
Copy link
Member

This commit classifies the Winch compiler in our documentation as a Tier 1 feature. This reflects the degree of work and investment put into it as well as the thorough quality vetting that has already happened in production. For Wasmtime this means that bugs in Winch are candidates for security releases as well.

This additionally refines our tiers-of-support documentation to reflect how not all compilers are required to support all features. Tables are added of per-architecture support for each compiler as well as refined wording of the various requirements of what is required for each tier as well.

This commit classifies the Winch compiler in our documentation as a Tier
1 feature. This reflects the degree of work and investment put into it
as well as the thorough quality vetting that has already happened in
production. For Wasmtime this means that bugs in Winch are candidates
for security releases as well.

This additionally refines our tiers-of-support documentation to reflect
how not all compilers are required to support all features. Tables are
added of per-architecture support for each compiler as well as refined
wording of the various requirements of what is required for each tier as
well.
@alexcrichton alexcrichton requested a review from a team as a code owner May 9, 2025 01:45
@alexcrichton alexcrichton requested review from dicej and removed request for a team May 9, 2025 01:45
@alexcrichton
Copy link
Member Author

cc @saulecabrera

@github-actions github-actions bot added the wasmtime:docs Issues related to Wasmtime's documentation label May 9, 2025
Comment on lines +415 to +423
* **Target**: tests are run in CI for this platform with a compiler, either
directly or via emulation. This ensures that all changes to Wasmtime are
tested against this target, and this additionally means that all applicable
Tier 1/2 features are tested on this target in CI.

* **Target**: Complete implementations for anything that's part of Tier 1. For
example all Tier 2 targets must implement all of the applicable Tier 1
WebAssembly proposals, and all Tier 2 features must be implemented on all Tier
1 targets.
Copy link
Member

@saulecabrera saulecabrera May 9, 2025

Choose a reason for hiding this comment

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

Small note on formatting: could these two be included under a single Target bullet point? e.g.,

  • Target
    • Tests are run ...
    • Complete implementation ...

In the rendered version, it took me a bit to understand the current formatting, but if I'm understanding correctly, they are meant to be two separate points under the same Target classification (similarly below, with Maintenance).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wasmtime:docs Issues related to Wasmtime's documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants