Keith Winstein

WastParser handling of `(module binary)` and `(module quote)`

1

WastParser seems to: - allow `(module binary "...")` in `ParseModule` (even though this is not valid syntax for the Wasm text format; it's only valid for scripts), and - forbid...

Remove intrusive_list (NFC)

1

This PR replaces the `intrusive_list` class with a std::list of unique_ptr's, trying to keep everything else mostly the same.

wasm2c roadmap ideas

9

Now that wasm2c has almost caught up to the current Wasm spec, maybe it's a good time to brainstorm about the roadmap from here and see what everything else thinks...

Enforce disabled features in WastParser for extended-const and memory64?

1

In some cases, WastParser rejects constructs that use disabled features (e.g. multi-memory, exceptions) and in some cases it appears to allow them (e.g. extended-const, memory64) even if disabled. (As seen...

[test] Test tail-call to imported function

1

There aren't any tests yet that use imports or attempt to tail-call an imported function -- there probably should be one (this would have caught a bug in wasm2c, https://github.com/WebAssembly/wabt/pull/2420).

wasmparser accepts some malformed modules

5

In the legacy-exceptions proposal, this module is malformed: ```wasm (func catch_all) ``` but wasmparser accepts it. This prevents passing some of the `assert_malformed` tests in the legacy-exceptions tests. (The resulting...