Fix wasm2js memory import in case it is minified (#3113)

It was hardcoded as "env.memory", which is usually correct. But if we minify
import names, as in -O3 in emscripten, we need to use the minified name.

Note how in the test it now emits

var memory = env.a;
for the import.

Fixes emscripten-core/emscripten#12123

This was not noticed earlier since that import is only used in memory
growth. The tests that would catch it are wasm2js3.test*memory_growth*
but we only run wasm2js1 on CI. I'll add testing after this lands.

1 files changed, 1 insertions, 1 deletions

diff --git a/src/wasm2js.h b/src/wasm2js.h
index ab093dd03..ac0268e5b 100644
--- a/src/wasm2js.h
+++ b/src/wasm2js.h
@@ -382,7 +382,7 @@ Ref Wasm2JSBuilder::processWasm(Module* wasm, Name funcName) {
       theVar,
       "memory",
       ValueBuilder::makeDot(ValueBuilder::makeName(ENV),
-                            ValueBuilder::makeName("memory")));
+                            ValueBuilder::makeName(wasm->memory.base)));
   }
   // for emscripten, add a table import - otherwise we would have
   // FUNCTION_TABLE be an upvar, and not as easy to be minified.