Replace Module["asm"] with wasmExports global

This brings the regular runtime closer `MINIMAL_RUNTIME` which uses
the global `asm` to refer to exports object.

As a followup we should consider merging these two symbols.

Author: sbc100

ChangeLog.md

@@ -20,6 +20,9 @@ See docs/process.md for more on how version tagging works.
 
 3.1.43 (in development)
 -----------------------
+- The `asm` proporty of the Module object (which olds the exports of the wasm
+  module) has been removed by default.  Internally, this is now known as
+  `wasmExports`.
 - Handling i64 arguments and return values in JS functions is now much simpler
   with the new `__i53abi` decorator.  When this is set to true, i64 values are
   automatically converted to JS numbers (i53) at the JS boundary.  Parameters

emscripten.py

@@ -771,7 +771,7 @@ def install_wrapper(sym):
       # With assertions disabled the wrapper will replace the global var and Module var on
       # first use.
       wrapper += f'''function() {{
-  return ({mangled} = {exported}Module['asm']['{name}']).apply(null, arguments);
+  return ({mangled} = {exported}wasmExports['{name}']).apply(null, arguments);
 }};
 '''
     else:

site/source/docs/optimizing/Module-Splitting.rst

@@ -124,7 +124,7 @@ included in the profile.
 Here’s the function to write the profile and our new main function::
 
   EM_JS(void, write_profile, (), {
-    var __write_profile = Module['asm']['__write_profile'];
+    var __write_profile = wasmExports['__write_profile'];
     if (__write_profile) {
 
       // Get the size of the profile and allocate a buffer for it.
@@ -338,7 +338,7 @@ be called either.
 
 When eagerly instantiating the secondary module, the imports object should be::
 
-  {'primary': Module['asm']}
+  {'primary': wasmExports}
 
 Debugging
 ---------

src/library_async.js

@@ -267,9 +267,9 @@ mergeInto(LibraryManager.library, {
     getDataRewindFunc: function(ptr) {
       var id = {{{ makeGetValue('ptr', C_STRUCTS.asyncify_data_s.rewind_id, 'i32') }}};
       var name = Asyncify.callStackIdToName[id];
-      var func = Module['asm'][name];
+      var func = wasmExports[name];
 #if RELOCATABLE
-      // Exported functions in side modules are not listed in `Module['asm']`,
+      // Exported functions in side modules are not listed in `wasmExports`,
       // So we should use `resolveGlobalSymbol` helper function, which is defined in `library_dylink.js`.
       if (!func) {
         func = resolveGlobalSymbol(name, false).sym;
@@ -522,8 +522,8 @@ mergeInto(LibraryManager.library, {
   _load_secondary_module__sig: 'v',
   _load_secondary_module: async function() {
     // Mark the module as loading for the wasm module (so it doesn't try to load it again).
-    Module['asm']['load_secondary_module_status'].value = 1;
-    var imports = {'primary': Module['asm']};
+    wasmExports['load_secondary_module_status'].value = 1;
+    var imports = {'primary': wasmExports};
     // Replace '.wasm' suffix with '.deferred.wasm'.
     var deferred = wasmBinaryFile.slice(0, -5) + '.deferred.wasm';
     await new Promise((resolve) => {

src/library_exceptions.js

@@ -336,7 +336,7 @@ var LibraryExceptions = {
 #if RELOCATABLE
     return ___cpp_exception; // defined in library.js
 #else
-    return Module['asm']['__cpp_exception'];
+    return wasmExports['__cpp_exception'];
 #endif
   },
 

src/library_exports.js

@@ -16,7 +16,7 @@ mergeInto(LibraryManager.library, {
     var exportedFunc = asm[name];
 #else
     // In regular runtime, exports are available on the Module object.
-    var exportedFunc = Module['asm'][name];
+    var exportedFunc = wasmExports[name];
 #endif
     if (exportedFunc) {
       // Record the created function pointer to each function object,

src/postamble_minimal.js

@@ -142,7 +142,7 @@ WebAssembly.instantiate(Module['wasm'], imports).then((output) => {
 
 #if !LibraryManager.has('library_exports.js') && !EMBIND
   // If not using the emscripten_get_exported_function() API or embind, keep the
-  // 'asm' exports variable in local scope to this instantiate function to save
+  // `asm` exports variable in local scope to this instantiate function to save
   // code size.  (otherwise access it without to export it to outer scope)
   var
 #endif

src/preamble.js

@@ -53,6 +53,7 @@ if (typeof WebAssembly != 'object') {
 // Wasm globals
 
 var wasmMemory;
+var wasmExports;
 
 #if SHARED_MEMORY
 // For sending to workers.
@@ -468,7 +469,7 @@ function abort(what) {
 #if WASM_EXCEPTIONS == 1
   // See above, in the meantime, we resort to wasm code for trapping.
   //
-  // In case abort() is called before the module is initialized, Module['asm']
+  // In case abort() is called before the module is initialized, wasmExports
   // and its exported '__trap' function is not available, in which case we throw
   // a RuntimeError.
   //
@@ -524,7 +525,7 @@ function createExportWrapper(name, fixedasm) {
     var displayName = name;
     var asm = fixedasm;
     if (!fixedasm) {
-      asm = Module['asm'];
+      asm = wasmExports;
     }
     assert(runtimeInitialized, 'native function `' + displayName + '` called before runtime initialization');
 #if EXIT_RUNTIME
@@ -714,7 +715,7 @@ var splitModuleProxyHandler = {
       throw new Error('Placeholder function "' + prop + '" should not be called when using JSPI.');
 #else
       err('placeholder function called: ' + prop);
-      var imports = {'primary': Module['asm']};
+      var imports = {'primary': wasmExports};
       // Replace '.wasm' suffix with '.deferred.wasm'.
       var deferred = wasmBinaryFile.slice(0, -5) + '.deferred.wasm'
       loadSplitModule(deferred, imports, prop);
@@ -979,18 +980,18 @@ function createWasm() {
     exports = applySignatureConversions(exports);
 #endif
 
-    Module['asm'] = exports;
+    wasmExports = exports;
 
 #if PTHREADS
 #if MAIN_MODULE
-    registerTLSInit(Module['asm']['_emscripten_tls_init'], instance.exports, metadata);
+    registerTLSInit(wasmExports['_emscripten_tls_init'], instance.exports, metadata);
 #else
-    registerTLSInit(Module['asm']['_emscripten_tls_init']);
+    registerTLSInit(wasmExports['_emscripten_tls_init']);
 #endif
 #endif
 
 #if !IMPORTED_MEMORY
-    wasmMemory = Module['asm']['memory'];
+    wasmMemory = wasmExports['memory'];
 #if ASSERTIONS
     assert(wasmMemory, "memory not found in wasm exports");
     // This assertion doesn't hold when emscripten is run in --post-link
@@ -1005,7 +1006,7 @@ function createWasm() {
 #endif
 
 #if !RELOCATABLE
-    wasmTable = Module['asm']['__indirect_function_table'];
+    wasmTable = wasmExports['__indirect_function_table'];
 #if ASSERTIONS && !PURE_WASI
     assert(wasmTable, "table not found in wasm exports");
 #endif
@@ -1019,11 +1020,11 @@ function createWasm() {
 #endif
 
 #if hasExportedSymbol('__wasm_call_ctors')
-    addOnInit(Module['asm']['__wasm_call_ctors']);
+    addOnInit(wasmExports['__wasm_call_ctors']);
 #endif
 
 #if hasExportedSymbol('__wasm_apply_data_relocs')
-    __RELOC_FUNCS__.push(Module['asm']['__wasm_apply_data_relocs']);
+    __RELOC_FUNCS__.push(wasmExports['__wasm_apply_data_relocs']);
 #endif
 
 #if ABORT_ON_WASM_EXCEPTIONS

src/shell.js

@@ -492,6 +492,7 @@ assert(typeof Module['readAsync'] == 'undefined', 'Module.readAsync option was r
 assert(typeof Module['readBinary'] == 'undefined', 'Module.readBinary option was removed (modify readBinary in JS)');
 assert(typeof Module['setWindowTitle'] == 'undefined', 'Module.setWindowTitle option was removed (modify setWindowTitle in JS)');
 assert(typeof Module['TOTAL_MEMORY'] == 'undefined', 'Module.TOTAL_MEMORY has been renamed Module.INITIAL_MEMORY');
+{{{ makeRemovedModuleAPIAssert('arm', 'wasmExports') }}}
 {{{ makeRemovedModuleAPIAssert('read', 'read_') }}}
 {{{ makeRemovedModuleAPIAssert('readAsync') }}}
 {{{ makeRemovedModuleAPIAssert('readBinary') }}}

test/optimizer/applyDCEGraphRemovals-output.js

@@ -18,21 +18,21 @@ var expD5 = asm["expD5"];
 var expD6;
 
 var expI1 = Module["expI1"] = function() {
- return (expI1 = Module["expI1"] = Module["asm"]["expI1"]).apply(null, arguments);
+ return (expI1 = Module["expI1"] = wasmExports["expI1"]).apply(null, arguments);
 };
 
 var expI2 = Module["expI2"] = function() {
- return (expI2 = Module["expI2"] = Module["asm"]["expI2"]).apply(null, arguments);
+ return (expI2 = Module["expI2"] = wasmExports["expI2"]).apply(null, arguments);
 };
 
 var expI3 = Module["expI3"] = function() {
- return (expI3 = Module["expI3"] = Module["asm"]["expI3"]).apply(null, arguments);
+ return (expI3 = Module["expI3"] = wasmExports["expI3"]).apply(null, arguments);
 };
 
 var expI4;
 
 var expI5 = function() {
- return (expI5 = Module["asm"]["expI5"]).apply(null, arguments);
+ return (expI5 = wasmExports["expI5"]).apply(null, arguments);
 };
 
 var expI6;