Skip to content

wasmparser(CM+GC): Add support for lowering variants, options, and results to GC types #2217

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

Merged
merged 2 commits into from
Jun 3, 2025

Conversation

fitzgen
Copy link
Member

@fitzgen fitzgen commented Jun 3, 2025

This completes prototype support for GC in the component model by filling out the lowering of sum types (variants, options, and results) to GC types.

…sults to GC types

This completes prototype support for GC in the component model by filling out
the lowering of sum types (variants, options, and results) to GC types.
@fitzgen fitzgen requested a review from a team as a code owner June 3, 2025 20:12
@fitzgen fitzgen requested review from alexcrichton and removed request for a team June 3, 2025 20:12
@alexcrichton
Copy link
Member

Thanks for writing this up and the proposal upstream as well, cleared up all possible questions I'd have about variants 👍

@alexcrichton alexcrichton enabled auto-merge June 3, 2025 20:34
@alexcrichton alexcrichton added this pull request to the merge queue Jun 3, 2025
@fitzgen fitzgen removed this pull request from the merge queue due to a manual request Jun 3, 2025
@fitzgen fitzgen added this pull request to the merge queue Jun 3, 2025
@fitzgen
Copy link
Member Author

fitzgen commented Jun 3, 2025

(Whoops I thought your approval was before the second commit, nvm)

Merged via the queue into bytecodealliance:main with commit 18c1c95 Jun 3, 2025
32 checks passed
@fitzgen fitzgen deleted the cm-gc-lower-sum-types branch June 3, 2025 20:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants