Wasm: Parsing for try/catch/throw

NEW
Unassigned

Status

()

Core
JavaScript Engine: JIT
P3
normal
a year ago
11 months ago

People

(Reporter: dmajor (offline), Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Reporter)

Description

a year ago
WIP patches for parsing, AST, text, etc. of try/catch/throw operators.

Currently these are just structural placeholders with no actual exception handling wired up.

For now, `throw` is more or less a `drop` that only accepts I32; `try` is like an `if` with no condition; and `catch` is like an `else` but with an element of `getLocal` for the catch-parameter.
(Reporter)

Comment 1

a year ago
Created attachment 8832340 [details] [diff] [review]
Part 1 - stubs for various future functions

This is roughly based on all the places where we have code for `growMemory`.
(Reporter)

Comment 2

a year ago
Created attachment 8832341 [details] [diff] [review]
Part 2 - throw keyword

This works reasonably well and includes some tests.
(Reporter)

Comment 3

a year ago
Created attachment 8832342 [details] [diff] [review]
Part 3 - try and catch (incomplete, buggy)
(Reporter)

Updated

a year ago
Attachment #8832342 - Attachment is patch: true
Not sure what the deadline is. Marking as P3. Feel free to update.
Priority: -- → P3
You need to log in before you can comment on or make changes to this bug.