Skip to content

test: run the wasm2c spec tests with memory64 on Windows - #2905

Merged
sbc100 merged 1 commit into
WebAssembly:mainfrom
natify-sh:wasm2c-windows-memory64-skip
Oct 22, 2026
Merged

sbc100 merged 1 commit into
WebAssembly:mainfrom
natify-sh:wasm2c-windows-memory64-skip

Conversation

@o-murphy

Copy link
Copy Markdown
Contributor

run-spec-wasm2c.py skips a test on Windows when memory64 is enabled. That skip was written
when memory64 was opt-in (--enable-memory64, #2086). Since #2837 memory64 is on by default
(--disable-memory64), so the condition holds for almost every test and 260 of the 262 wasm2c
tests are skipped on windows-latest: the C that wasm2c writes is not compiled by cl.exe.

This removes the skip. Split out of #2901 at @sbc100's request.

@sbc100 sbc100 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm if the CI passes

@sbc100
sbc100 marked this pull request as ready for review October 22, 2026 20:33
@sbc100
sbc100 enabled auto-merge (squash) October 22, 2026 20:33
@sbc100
sbc100 merged commit aeba296 into WebAssembly:main Oct 22, 2026
17 checks passed
@o-murphy
o-murphy deleted the wasm2c-windows-memory64-skip branch October 22, 2026 21:27
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