Support WASM version 0x1 Binaries |
||||||
Issue descriptionThe WebAssembly implementation in Chrome should also accept version 0x1 binaries.
,
Feb 9 2017
,
Feb 9 2017
Please add the OSs that apply. Thanks.
,
Feb 9 2017
Please apply appropriate OS labels. Thank you.
,
Feb 9 2017
,
Feb 10 2017
Your change meets the bar and is auto-approved for M57. Please go ahead and merge the CL to branch 2987 manually. Please contact milestone owner if you have questions. Owners: amineer@(clank), cmasso@(bling), ketakid@(cros), govind@(desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Feb 10 2017
Please merge your change to M57 branch 2987 before 5:00 PM PT, Friday 02/10 so we can take it in for next week beta release. Thank you.
,
Feb 10 2017
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/208c1202a1941e95e3fa70fc1d9c40644d67f989 commit 208c1202a1941e95e3fa70fc1d9c40644d67f989 Author: titzer <titzer@chromium.org> Date: Fri Feb 10 21:30:27 2017 Merged: [wasm] Accept version 0x1 binaries. Revision: fa7d1f8f75e340d917831ef74f401154b0a90cc7 BUG=chromium:575167, chromium:690281 LOG=N NOTRY=true NOPRESUBMIT=true NOTREECHECKS=true R=bradnelson@chromium.org, hablich@chromium.org Review-Url: https://codereview.chromium.org/2692563002 Cr-Commit-Position: refs/branch-heads/5.7@{#96} Cr-Branched-From: 975e9a320b6eaf9f12280c35df98e013beb8f041-refs/heads/5.7.492@{#1} Cr-Branched-From: 8d76f0e3465a84bbf0bceab114900fbe75844e1f-refs/heads/master@{#42426} [modify] https://crrev.com/208c1202a1941e95e3fa70fc1d9c40644d67f989/src/wasm/module-decoder.cc [modify] https://crrev.com/208c1202a1941e95e3fa70fc1d9c40644d67f989/src/wasm/wasm-module.h [modify] https://crrev.com/208c1202a1941e95e3fa70fc1d9c40644d67f989/test/inspector/debugger/wasm-scripts-expected.txt [modify] https://crrev.com/208c1202a1941e95e3fa70fc1d9c40644d67f989/test/mjsunit/wasm/incrementer.wasm [modify] https://crrev.com/208c1202a1941e95e3fa70fc1d9c40644d67f989/test/mjsunit/wasm/wasm-constants.js [modify] https://crrev.com/208c1202a1941e95e3fa70fc1d9c40644d67f989/test/unittests/value-serializer-unittest.cc
,
Feb 10 2017
Per comment #8, this is already merged to M57. |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by bugdroid1@chromium.org
, Feb 9 2017