Skip to content

Commit

Permalink
Update explainer for the new spec (#137)
Browse files Browse the repository at this point in the history
This updates the explainer text according to the new spec we agreed in
the 09-15-2020 CG meeting and discussions afterwards.

The following are modifications and clarifications we made after the
09-15-2020 CG meeting, and the relevant issue posts, if any:
https://github.com/WebAssembly/meetings/blob/master/main/2020/CG-09-15.md

- `catch_br` wasm renamed to `delegate` (#133)
- `rethrow` gains an immediate argument (#126)
- Removed dependences on the reference types proposal and the multivalue
  proposal. The multivalue proposal was previously listed as dependent
  because 1. `try` is basically a `block`, so it can have multivalue
  input/output 2. `br_on_exn` can extract multiple values from a
  `block`. We don't have `br_on_exn` anymore, and I'm not sure 1 is a
  strong enough reason to make it a dependence.
- Mention `rethrow` cannot rethrow exceptions caught by `unwind` (#142
  and #137)
- Mention some runtimes, especially web VMs, can attach stack traces to
  the exception object, implying stack traces are not required for all
  VMs
- Update label/validation rules for `delegate` and `rethrow` (#146)
- Finalize opcodes for `delegate` (0x18) and `catch_all` (0x19) (#145
  and #147)

I believe this resolves many previous issue threads, so I'll close them.
Please reopen them if you think there are things left for discussions in
those issues.

Resolves #113, resolves #126, resolves #127, resolves #128, resolves
#130, resolves #142, resolves #145, resolves #146, resolves #147.
  • Loading branch information
aheejin committed Feb 25, 2021
1 parent 4695ec3 commit 176d5c3
Show file tree
Hide file tree
Showing 2 changed files with 242 additions and 179 deletions.
2 changes: 0 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,6 @@ for adding exception handling to WebAssembly.
* See the [proposal overview](proposals/exception-handling/Exceptions.md) for a
summary of the proposal.

The repository is now based on the [reference types proposal](proposals/reference-types/Overview.md) and includes all respective changes.

Original README from upstream repository follows...

# spec
Expand Down
Loading

0 comments on commit 176d5c3

Please sign in to comment.