Skip to content

Add a few more miscellaneous tests#222

Merged
sunfishcode merged 8 commits intomasterfrom
more-tests
Feb 2, 2016
Merged

Add a few more miscellaneous tests#222
sunfishcode merged 8 commits intomasterfrom
more-tests

Conversation

@sunfishcode
Copy link
Member

No description provided.

@rossberg
Copy link
Member

rossberg commented Feb 2, 2016

lgtm

sunfishcode added a commit that referenced this pull request Feb 2, 2016
Add a few more miscellaneous tests
@sunfishcode sunfishcode merged commit be14e04 into master Feb 2, 2016
@sunfishcode sunfishcode deleted the more-tests branch February 2, 2016 17:52
ngzhian added a commit to ngzhian/spec that referenced this pull request Nov 4, 2021
* Refactor SIMD operators

* Common submodule for different SIMD lane shapes

* Tweak the submodule representation

* Rename modules, move module into Make

* MakeFloat takes a Float.S

* Better name for MakeFloat parameter
dhil pushed a commit to dhil/webassembly-spec that referenced this pull request Mar 2, 2023
* valid-try-catch: Typesetting and notation like in formal-overview.

* Fix todo:: Prose for valid-try-catch.
dhil pushed a commit to dhil/webassembly-spec that referenced this pull request Mar 2, 2023
… everywhere. (WebAssembly#241)

This addresses additional review comments to PR WebAssembly#222, that were made after it was merged.
The last review comment in the discussion suggests to adjust all validation labels to use label types instead of just result types.
Should address all occurrences of validation labels.

Additionally adds a boolean catch_label to control frames in the validation algorithm,
and some related functionality, fixing the cases for opcodes `catch` and `catch_all`.

* Apply suggestions from code review

Co-authored-by: Andreas Rossberg <rossberg@mpi-sws.org>
Co-authored-by: Heejin Ahn <aheejin@gmail.com>

* Reverting changes to typing of CAUGHTadm.

Changes to this rule are now done in PR WebAssembly#244
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants