Typed continuations: nocont and cont basic heap types#6468
Typed continuations: nocont and cont basic heap types#6468tlively merged 6 commits intoWebAssembly:mainfrom
Conversation
|
The |
|
Fair enough, I'm happy to keep the old name. I've undone the renaming and updated the PR description. |
tlively
left a comment
There was a problem hiding this comment.
LGTM, but for more test coverage it might be a good idea to add continuation types to TestHeapTypeRelations in test/gtest/type-builder.cpp as well.
|
Good call, I actually caught a place I had forgotten to update this way! |
|
Just FYI, this was the last missing piece to add basic support for the core of the proposal to binaryen. Only I'd therefore prioritize implementing validation for the already added instructions first. Can the new wat/wast parser now be used for spec tests, or is that going to be the case soon-ish? I'm asking because I'm hoping to re-use our existing spec tests when adding validation, rather than having to transform them into binaryen's text format. |
|
No support for running spec tests with the new parser yet, but I hope to be done with that within a couple weeks. |
|
@frank-emrich, the new parser is now used for top-level modules in wasm-shell, so you should try running your spec tests now. It's possible you'll run into some other limitations, but at least there should be fewer now. I plan to keep working toward being able to run the full upstream test suite. |
|
That's great, thanks for letting me know! |
This PR is part of a series that adds basic support for the typed continuations/wasmfx proposal.
This particular types adds
contandnocontas top and bottom types for continuation types, completely analogous tofuncandnofuncfor function types (also:exnandnoexn).