Skip to content

SIMD roundtrip test failure - #1

Merged
sagacity merged 2 commits into
datatrash:mainfrom
revivalizer:simd-roundtrip-test
Jul 22, 2026
Merged

sagacity merged 2 commits into
datatrash:mainfrom
revivalizer:simd-roundtrip-test

Conversation

@revivalizer

@revivalizer revivalizer commented Jul 10, 2026 •

Copy link
Copy Markdown
Contributor

During development of an intro I observed mashi incorrectly decompressing a wasm file.

I attached two commits:

  • The first commit adds a test on the failing wasm, which fails cargo test --release. If you amend the trace with the LEB accumulator it should be quite clear that the rust and wasm decompressors diverge. I added the whole wasm because it provides good coverage on SIMD, but I could provide a smaller repro case if you prefer.
  • The second commit adds a fix proposed by Claude. Feel free to ignore, or let me know if you prefer a PR without it.

@sagacity

Copy link
Copy Markdown
Member

Thanks for reporting! I'll try to follow up in a few days.

@sagacity
sagacity merged commit 289f79d into datatrash:main Jul 22, 2026
5 checks passed
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