New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add an initial test for wasm multi-value. #17314
Conversation
The last test appears to work in the V8 shell when passing CC @rossberg |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests look correct to me, but maybe you meant to include further coverage. To pass V8 flags from Chrome invoked on the command line, use --js-flags="--experimental-wasm-mv"
.
assert_true(Array.isArray(swapped)); | ||
assert_equals(Object.getPrototypeOf(swapped), Array.prototype); | ||
assert_array_equals(swapped, [4.2, 7]); | ||
}, "multiple return values from js to wasm"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't understand how this is JS to Wasm. Did you mean to make further changes to this test?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It isn't, indeed. I removed this part; I'll submit a JS to wasm test at some later point.
7ca158a
to
cddc97b
Compare
No description provided.