[spec] Disagreement between the tests and the spec on the import of tags
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
- Clearly specified
- Activity status
- Stale
- Tech stack
- wasm
- Domain
- compilers
Research direction
Compare the import grammar in sections 6.6.3 and 6.6.8 of the wasm-3.0 specification with the example in test/core/tag.wast. Reconcile the grammar and test so the documented tag import syntax is consistent, then verify the relevant core tests.
Written by the indexing model from the issue text.
Description
The text format for importing tags (section 6.6.3, wasm-3.0) is (import mod:name nm:name (tag id? tt:tag)), where tag is defined as (tag id? x,I':typeuse) (section 6.6.8). A tag import, would thus require twice the tag keyword. It is not what is done in the test core/tag.wat :
(import "test" "t3" (tag $t1 (param i32 f32)))
The tt : tag could be replaced by a typeuse or the definition of importdesc_I could be of the form ... | tag_I.
- Dominant language
- WebAssembly
- Stars
- 3.5k
- Forks
- 540
- Avg merge
- 1d 10h
- Merged PRs (30d)
- 16
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 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 5 days 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 31 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
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
WebAssembly/spec#2196 ·
Maintainers usually reply within 1 day
All issues in WebAssembly/spec
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
jaseci-labs/jac#9934 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
rescript-lang/rescript#8763 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
flagos-ai/FlagTree#1344 · 1 comment ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 66/100
Maintainers usually reply within 5 days