Don't emit `module definition` or `module instance` unless present in input when generating `.bin.wast`
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 45/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Stale
- Tech stack
- wasm
- Domain
- compilers
Research direction
Start in arrange.ml, where PR #1796 changed the reference interpreter's generation of .bin.wast output. Trace how the original source is represented and how module definition and module instance commands are emitted. Done means those commands appear in generated output only when they were present in the input, without breaking downstream .bin.wast consumers.
Written by the indexing model from the issue text.
Description
The PR #1796 changed the reference interpreter to generate these new commands by default in arrange.ml. Unfortunately this breaks downstream clients that consume .bin.wast.
Would it be possible to only emit module definition and module instance if they were in the original source?
- Dominant language
- WebAssembly
- Stars
- 3.5k
- Forks
- 539
- Avg merge
- 10h 24m
- Merged PRs (30d)
- 11
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from WebAssembly/spec
-
Difficulty 4/5 3-5 days Newbie friendliness 54/100
WebAssembly/spec#2265 ·
Maintainers usually reply within 1 day
-
Difficulty 5/5 Over a week Newbie friendliness 35/100
WebAssembly/spec#2258 · 4 comments · 1 reaction ·
Maintainers usually reply within 1 day
-
[js-api] A mutable global import allocates a const global before LinkErrorPossibly taken @chicoxyzzy claimed this 1 day ago. Open
Difficulty 3/5 1-2 days Newbie friendliness 58/100
WebAssembly/spec#2253 ·
Maintainers usually reply within 1 day
-
[spectec] Wasm 1.0: `$instantiate` missing premisesPossibly taken @rossberg claimed this 27 days ago. Open
Difficulty 3/5 1-2 days Newbie friendliness 68/100
WebAssembly/spec#2245 ·
Maintainers usually reply within 1 day
-
Difficulty 4/5 3-5 days Newbie friendliness 45/100
WebAssembly/spec#2235 · 9 comments ·
Maintainers usually reply within 1 day
All issues in WebAssembly/spec
Similar issues
-
area:prove bug
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
Chelis-Lang/chelis#3317 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
microsoft/TypeScript#64661 ·
Maintainers usually reply within 1 day
-
good first issue
Difficulty 2/5 1-3 hours Newbie friendliness 66/100
Sim20004/teapot-lang#248 ·