Closed Bug 1254984 Opened 5 years ago Closed 5 years ago

Build experimental text representation of the WebAssembly binary code

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla48
Tracking Status
firefox48 --- fixed

People

(Reporter: yury, Assigned: yury)

References

Details

Attachments

(2 files, 7 obsolete files)

During debugging it will be useful to show something in text form for the running WebAssembly code. The text format is not finalized yet (https://github.com/WebAssembly/design/blob/master/TextFormat.md), but already can have running Wasm code in the SpiderMonkey. It is expected to have some function that will convert binary code produced by wasmTextToBinary into experimental text form.
Attached patch wip1 (obsolete) — Splinter Review
Attached patch wip2 (obsolete) — Splinter Review
Attachment #8728744 - Attachment is obsolete: true
Attached patch wip3 (obsolete) — Splinter Review
Attachment #8728762 - Attachment is obsolete: true
Binary to wast/S-expression format conversion. Parsing logic is based on js/src/asmjs/Wasm.cpp and shall be kept in sync.
Attachment #8729042 - Attachment is obsolete: true
Attachment #8729277 - Flags: review?(luke)
Fixes some sm style checks -- passing try server https://treeherder.mozilla.org/#/jobs?repo=try&revision=65df4b2773c9
Attachment #8729277 - Attachment is obsolete: true
Attachment #8729277 - Flags: review?(luke)
Attachment #8729689 - Flags: review?(luke)
More tests and fixed br_table display

https://treeherder.mozilla.org/#/jobs?repo=try&revision=d0dc74fa42f1
Attachment #8729689 - Attachment is obsolete: true
Attachment #8729689 - Flags: review?(luke)
Attachment #8729761 - Flags: review?(luke)
Comment on attachment 8729761 [details] [diff] [review]
Experimental text representation of WebAssembly binary code

Great work busting this out!  The tests look good too.  The iterator refactoring should reduce the amount of duplication between this and Wasm.cpp.  With permission, I'll fix the nits inline and land the patch.
Attachment #8729761 - Flags: review?(luke) → review+
https://hg.mozilla.org/mozilla-central/rev/3e871f2d5b4b
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
See Also: → 1274618
You need to log in before you can comment on or make changes to this bug.