Skip to content
This repository has been archived by the owner on Jun 16, 2020. It is now read-only.

Improve support for reftypes #168

Merged
merged 2 commits into from
Jan 9, 2020
Merged

Improve support for reftypes #168

merged 2 commits into from
Jan 9, 2020

Conversation

eqrion
Copy link
Contributor

@eqrion eqrion commented Jan 6, 2020

There's been a couple of recent changes to the reference types proposal that are required for CL/Spidermonkey.

The reftypes proposal adds a typed select operator to simplify type checking
for reference types. The existing select operator is henceforth limited to
operate on the original MVP types only.

Spec change: WebAssembly/reference-types#43
@yurydelendik yurydelendik merged commit fb3185b into bytecodealliance:master Jan 9, 2020
@yurydelendik
Copy link
Collaborator

Thank you for the patch!

Are there are tests in https://github.com/WebAssembly/testsuite we can enable?

@eqrion
Copy link
Contributor Author

eqrion commented Jan 13, 2020

Thank you for the patch!

Are there are tests in https://github.com/WebAssembly/testsuite we can enable?

Good question. It looks like most reference-types tests are currently passing. I tested out the ones still being skipped and found that two are now passing. I'll put up a PR for them.

yurydelendik pushed a commit that referenced this pull request Jan 13, 2020
These tests are now passing, possibly because of #168.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants