Commit 74f5de4f authored by Thibaud Michaud's avatar Thibaud Michaud Committed by Commit Bot

[wasm][mv] Enable WebAssembly multi-value

Intent to Ship thread on blink-dev:
https://groups.google.com/a/chromium.org/g/blink-dev/c/K4-P5rq7FbY

Spec repository:
https://github.com/WebAssembly/multi-value/

WebAssembly multi-value is ready to ship. The proposal is now merged
into the core WebAssembly spec, and is implemented in both TurboFan and
Liftoff.

R=ecmziegler@chromium.org

Bug: chromium:1097717
Change-Id: Id53ee318cacae1bc9dd9d9611f9223de594a99b7
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2257225Reviewed-by: 's avatarEmanuel Ziegler <ecmziegler@chromium.org>
Commit-Queue: Thibaud Michaud <thibaudm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#68459}
parent 5ef3e899
...@@ -55,12 +55,6 @@ ...@@ -55,12 +55,6 @@
/* Staged in v7.8. */ \ /* Staged in v7.8. */ \
V(reftypes, "reference type opcodes", false) \ V(reftypes, "reference type opcodes", false) \
\ \
/* Multi-value proposal. */ \
/* https://github.com/WebAssembly/multi-value */ \
/* V8 side owner: thibaudm */ \
/* Staged in v8.0. */ \
V(mv, "multi-value support", false) \
\
/* Threads proposal. */ \ /* Threads proposal. */ \
/* https://github.com/webassembly/threads */ \ /* https://github.com/webassembly/threads */ \
/* NOTE: This is enabled via chromium flag on desktop systems since v7.4 */ \ /* NOTE: This is enabled via chromium flag on desktop systems since v7.4 */ \
...@@ -92,7 +86,14 @@ ...@@ -92,7 +86,14 @@
/* V8 side owner: binji */ \ /* V8 side owner: binji */ \
/* Shipped in v7.5. */ \ /* Shipped in v7.5. */ \
/* ITS: https://groups.google.com/forum/#!topic/v8-users/zM05lYEBVog */ \ /* ITS: https://groups.google.com/forum/#!topic/v8-users/zM05lYEBVog */ \
V(bulk_memory, "bulk memory opcodes", true) V(bulk_memory, "bulk memory opcodes", true) \
\
/* Multi-value proposal. */ \
/* https://github.com/WebAssembly/multi-value */ \
/* V8 side owner: thibaudm */ \
/* Shipped in v8.5. */ \
/* ITS: https://groups.google.com/g/v8-users/c/pv2E4yFWeF0 */ \
V(mv, "multi-value support", true)
// Combination of all available wasm feature flags. // Combination of all available wasm feature flags.
#define FOREACH_WASM_FEATURE_FLAG(V) \ #define FOREACH_WASM_FEATURE_FLAG(V) \
......
...@@ -51,8 +51,7 @@ class TestCase(testcase.D8TestCase): ...@@ -51,8 +51,7 @@ class TestCase(testcase.D8TestCase):
for proposal in proposal_flags: for proposal in proposal_flags:
if os.sep.join(['proposals', proposal['name']]) in self.path: if os.sep.join(['proposals', proposal['name']]) in self.path:
return proposal['flags'] return proposal['flags']
# TODO(thibaudm): Remove the flag once multi-value is shipped in V8. return []
return ['--experimental-wasm-mv']
def GetSuite(*args, **kwargs): def GetSuite(*args, **kwargs):
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment