Skip to content

Comments

[2/3] [Rust] Host changes for RawModuleDefV10#4105

Merged
Shubham8287 merged 14 commits intomasterfrom
shub/raw-def-host
Jan 28, 2026
Merged

[2/3] [Rust] Host changes for RawModuleDefV10#4105
Shubham8287 merged 14 commits intomasterfrom
shub/raw-def-host

Conversation

@Shubham8287
Copy link
Contributor

Description of Changes

Host side changes for Wasm to work with RawModuleDefV10 along with prior versions.
Based on ##4098

API and ABI breaking changes

NA

Expected complexity level and risk

1.5

Testing

Existing test will cover for any regression.
Not yet functional for RawModuleDefV10, so can not be tested now.

@Shubham8287 Shubham8287 changed the base branch from shub/raw-defination to master January 28, 2026 13:05
@Shubham8287 Shubham8287 enabled auto-merge January 28, 2026 13:16
@Shubham8287 Shubham8287 removed the request for review from joshua-spacetime January 28, 2026 13:24
@Shubham8287 Shubham8287 added this pull request to the merge queue Jan 28, 2026
Merged via the queue into master with commit fd3cdb3 Jan 28, 2026
27 of 28 checks passed
@Shubham8287 Shubham8287 deleted the shub/raw-def-host branch January 28, 2026 18:33
Shubham8287 added a commit that referenced this pull request Feb 10, 2026
github-merge-queue bot pushed a commit that referenced this pull request Feb 11, 2026
This reverts commit fd3cdb3.

# Description of Changes
Reverts #4105, We do not actually need `__describe_module_v10__`, as
original describe reducer returns `enum` - `crates:lib::RawModuleDef`
which is extended to add a variant for `V10`

# API and ABI breaking changes
NA
# Expected complexity level and risk
1
# Testing
Existing tests should be enough
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