diff options
author | Brion Vibber <brion@pobox.com> | 2019-08-30 14:20:48 -0700 |
---|---|---|
committer | Alon Zakai <azakai@google.com> | 2019-08-30 14:20:47 -0700 |
commit | c134c574e9e755a85806d42481c1b6d490d0d445 (patch) | |
tree | 67f50fb8957d177b8d84e01ab7e3eb4b290b0213 | |
parent | eace464174f1abefead072dc580244483e499af6 (diff) | |
download | binaryen-c134c574e9e755a85806d42481c1b6d490d0d445.tar.gz binaryen-c134c574e9e755a85806d42481c1b6d490d0d445.tar.bz2 binaryen-c134c574e9e755a85806d42481c1b6d490d0d445.zip |
Followup to workaround for minification of wasm2js mem init (#2318)
Emscripten's minifier mis-minifies a couple bits in the memory
init function that's used with wasm2js when not using an external
memory init file:
https://github.com/emscripten-core/emscripten/issues/8886
Previous fix worked around the bug in one place but failed to
account for another. Have now confirmed that it works with this
change in place.
Updated test cases to match.
-rw-r--r-- | src/wasm2js.h | 6 | ||||
-rw-r--r-- | test/wasm2js/address.2asm.js | 6 | ||||
-rw-r--r-- | test/wasm2js/dynamicLibrary.2asm.js | 6 | ||||
-rw-r--r-- | test/wasm2js/dynamicLibrary.2asm.js.opt | 6 | ||||
-rw-r--r-- | test/wasm2js/emscripten-grow-no.2asm.js | 6 | ||||
-rw-r--r-- | test/wasm2js/emscripten-grow-no.2asm.js.opt | 6 | ||||
-rw-r--r-- | test/wasm2js/emscripten-grow-yes.2asm.js | 6 | ||||
-rw-r--r-- | test/wasm2js/emscripten-grow-yes.2asm.js.opt | 6 | ||||
-rw-r--r-- | test/wasm2js/emscripten.2asm.js | 6 | ||||
-rw-r--r-- | test/wasm2js/emscripten.2asm.js.opt | 6 |
10 files changed, 30 insertions, 30 deletions
diff --git a/src/wasm2js.h b/src/wasm2js.h index 54a06578f..1526dbb89 100644 --- a/src/wasm2js.h +++ b/src/wasm2js.h @@ -2231,14 +2231,14 @@ void Wasm2JSGlue::emitMemory( function(mem) { var _mem = new Uint8Array(mem); return function(offset, s) { - var bytes; + var bytes, i; if (typeof Buffer === 'undefined') { bytes = atob(s); - for (var i = 0; i < bytes.length; i++) + for (i = 0; i < bytes.length; i++) _mem[offset + i] = bytes.charCodeAt(i); } else { bytes = Buffer.from(s, 'base64'); - for (var i = 0; i < bytes.length; i++) + for (i = 0; i < bytes.length; i++) _mem[offset + i] = bytes[i]; } } diff --git a/test/wasm2js/address.2asm.js b/test/wasm2js/address.2asm.js index 6c1261a4e..c811a1a2e 100644 --- a/test/wasm2js/address.2asm.js +++ b/test/wasm2js/address.2asm.js @@ -82,14 +82,14 @@ var assignasmFunc = ( function(mem) { var _mem = new Uint8Array(mem); return function(offset, s) { - var bytes; + var bytes, i; if (typeof Buffer === 'undefined') { bytes = atob(s); - for (var i = 0; i < bytes.length; i++) + for (i = 0; i < bytes.length; i++) _mem[offset + i] = bytes.charCodeAt(i); } else { bytes = Buffer.from(s, 'base64'); - for (var i = 0; i < bytes.length; i++) + for (i = 0; i < bytes.length; i++) _mem[offset + i] = bytes[i]; } } diff --git a/test/wasm2js/dynamicLibrary.2asm.js b/test/wasm2js/dynamicLibrary.2asm.js index d0a2d556f..51a3aaba1 100644 --- a/test/wasm2js/dynamicLibrary.2asm.js +++ b/test/wasm2js/dynamicLibrary.2asm.js @@ -50,14 +50,14 @@ var assignasmFunc = ( function(mem) { var _mem = new Uint8Array(mem); return function(offset, s) { - var bytes; + var bytes, i; if (typeof Buffer === 'undefined') { bytes = atob(s); - for (var i = 0; i < bytes.length; i++) + for (i = 0; i < bytes.length; i++) _mem[offset + i] = bytes.charCodeAt(i); } else { bytes = Buffer.from(s, 'base64'); - for (var i = 0; i < bytes.length; i++) + for (i = 0; i < bytes.length; i++) _mem[offset + i] = bytes[i]; } } diff --git a/test/wasm2js/dynamicLibrary.2asm.js.opt b/test/wasm2js/dynamicLibrary.2asm.js.opt index 6692be1cb..862c1992a 100644 --- a/test/wasm2js/dynamicLibrary.2asm.js.opt +++ b/test/wasm2js/dynamicLibrary.2asm.js.opt @@ -42,14 +42,14 @@ var assignasmFunc = ( function(mem) { var _mem = new Uint8Array(mem); return function(offset, s) { - var bytes; + var bytes, i; if (typeof Buffer === 'undefined') { bytes = atob(s); - for (var i = 0; i < bytes.length; i++) + for (i = 0; i < bytes.length; i++) _mem[offset + i] = bytes.charCodeAt(i); } else { bytes = Buffer.from(s, 'base64'); - for (var i = 0; i < bytes.length; i++) + for (i = 0; i < bytes.length; i++) _mem[offset + i] = bytes[i]; } } diff --git a/test/wasm2js/emscripten-grow-no.2asm.js b/test/wasm2js/emscripten-grow-no.2asm.js index f9aeda1b4..895a54fc2 100644 --- a/test/wasm2js/emscripten-grow-no.2asm.js +++ b/test/wasm2js/emscripten-grow-no.2asm.js @@ -44,14 +44,14 @@ var writeSegment = ( function(mem) { var _mem = new Uint8Array(mem); return function(offset, s) { - var bytes; + var bytes, i; if (typeof Buffer === 'undefined') { bytes = atob(s); - for (var i = 0; i < bytes.length; i++) + for (i = 0; i < bytes.length; i++) _mem[offset + i] = bytes.charCodeAt(i); } else { bytes = Buffer.from(s, 'base64'); - for (var i = 0; i < bytes.length; i++) + for (i = 0; i < bytes.length; i++) _mem[offset + i] = bytes[i]; } } diff --git a/test/wasm2js/emscripten-grow-no.2asm.js.opt b/test/wasm2js/emscripten-grow-no.2asm.js.opt index f9aeda1b4..895a54fc2 100644 --- a/test/wasm2js/emscripten-grow-no.2asm.js.opt +++ b/test/wasm2js/emscripten-grow-no.2asm.js.opt @@ -44,14 +44,14 @@ var writeSegment = ( function(mem) { var _mem = new Uint8Array(mem); return function(offset, s) { - var bytes; + var bytes, i; if (typeof Buffer === 'undefined') { bytes = atob(s); - for (var i = 0; i < bytes.length; i++) + for (i = 0; i < bytes.length; i++) _mem[offset + i] = bytes.charCodeAt(i); } else { bytes = Buffer.from(s, 'base64'); - for (var i = 0; i < bytes.length; i++) + for (i = 0; i < bytes.length; i++) _mem[offset + i] = bytes[i]; } } diff --git a/test/wasm2js/emscripten-grow-yes.2asm.js b/test/wasm2js/emscripten-grow-yes.2asm.js index f4821154d..15363502e 100644 --- a/test/wasm2js/emscripten-grow-yes.2asm.js +++ b/test/wasm2js/emscripten-grow-yes.2asm.js @@ -71,14 +71,14 @@ var writeSegment = ( function(mem) { var _mem = new Uint8Array(mem); return function(offset, s) { - var bytes; + var bytes, i; if (typeof Buffer === 'undefined') { bytes = atob(s); - for (var i = 0; i < bytes.length; i++) + for (i = 0; i < bytes.length; i++) _mem[offset + i] = bytes.charCodeAt(i); } else { bytes = Buffer.from(s, 'base64'); - for (var i = 0; i < bytes.length; i++) + for (i = 0; i < bytes.length; i++) _mem[offset + i] = bytes[i]; } } diff --git a/test/wasm2js/emscripten-grow-yes.2asm.js.opt b/test/wasm2js/emscripten-grow-yes.2asm.js.opt index f4821154d..15363502e 100644 --- a/test/wasm2js/emscripten-grow-yes.2asm.js.opt +++ b/test/wasm2js/emscripten-grow-yes.2asm.js.opt @@ -71,14 +71,14 @@ var writeSegment = ( function(mem) { var _mem = new Uint8Array(mem); return function(offset, s) { - var bytes; + var bytes, i; if (typeof Buffer === 'undefined') { bytes = atob(s); - for (var i = 0; i < bytes.length; i++) + for (i = 0; i < bytes.length; i++) _mem[offset + i] = bytes.charCodeAt(i); } else { bytes = Buffer.from(s, 'base64'); - for (var i = 0; i < bytes.length; i++) + for (i = 0; i < bytes.length; i++) _mem[offset + i] = bytes[i]; } } diff --git a/test/wasm2js/emscripten.2asm.js b/test/wasm2js/emscripten.2asm.js index 4ade4b36a..a32f3edd6 100644 --- a/test/wasm2js/emscripten.2asm.js +++ b/test/wasm2js/emscripten.2asm.js @@ -202,14 +202,14 @@ var writeSegment = ( function(mem) { var _mem = new Uint8Array(mem); return function(offset, s) { - var bytes; + var bytes, i; if (typeof Buffer === 'undefined') { bytes = atob(s); - for (var i = 0; i < bytes.length; i++) + for (i = 0; i < bytes.length; i++) _mem[offset + i] = bytes.charCodeAt(i); } else { bytes = Buffer.from(s, 'base64'); - for (var i = 0; i < bytes.length; i++) + for (i = 0; i < bytes.length; i++) _mem[offset + i] = bytes[i]; } } diff --git a/test/wasm2js/emscripten.2asm.js.opt b/test/wasm2js/emscripten.2asm.js.opt index af543f397..2aa92c316 100644 --- a/test/wasm2js/emscripten.2asm.js.opt +++ b/test/wasm2js/emscripten.2asm.js.opt @@ -183,14 +183,14 @@ var writeSegment = ( function(mem) { var _mem = new Uint8Array(mem); return function(offset, s) { - var bytes; + var bytes, i; if (typeof Buffer === 'undefined') { bytes = atob(s); - for (var i = 0; i < bytes.length; i++) + for (i = 0; i < bytes.length; i++) _mem[offset + i] = bytes.charCodeAt(i); } else { bytes = Buffer.from(s, 'base64'); - for (var i = 0; i < bytes.length; i++) + for (i = 0; i < bytes.length; i++) _mem[offset + i] = bytes[i]; } } |