Enable EM_JS in side modules

This works in a similar way to EM_ASM. See #18228.

Depends on WebAssembly/binaryen#5780

emcc.py

@@ -831,6 +831,7 @@ def process_dynamic_libs(dylibs, lib_dirs):
   for dylib in dylibs:
     exports = webassembly.get_exports(dylib)
     exports = set(e.name for e in exports)
+    exports = [utils.removeprefix(e, '__em_js__') for e in exports]
     settings.SIDE_MODULE_EXPORTS.extend(sorted(exports))
 
     imports = webassembly.get_imports(dylib)

emscripten.py

@@ -344,8 +344,6 @@ def emscript(in_wasm, out_wasm, outfile_js, memfile, js_syms):
           diagnostics.warning('em-js-i64', 'using 64-bit arguments in EM_JS function without WASM_BIGINT is not yet fully supported: `%s` (%s, %s)', em_js_func, c_sig, signature.params)
 
   if settings.SIDE_MODULE:
-    if metadata.emJsFuncs:
-      exit_with_error('EM_JS is not supported in side modules')
     logger.debug('emscript: skipping remaining js glue generation')
     return
 

src/library_dylink.js

@@ -178,7 +178,9 @@ var LibraryDylink = {
       '__wasm_call_ctors',
       '__start_em_asm',
       '__stop_em_asm',
-    ].includes(symName)
+      '__start_em_js',
+      '__stop_em_js',
+    ].includes(symName) || symName.startsWith('__em_js__')
 #if SPLIT_MODULE
         // Exports synthesized by wasm-split should be prefixed with '%'
         || symName[0] == '%'
@@ -779,6 +781,29 @@ var LibraryDylink = {
             start = HEAPU8.indexOf(0, start) + 1;
           }
         }
+
+        function addEmJs(name, sig, body) {
+          sig = sig.slice(1, -1).split(',');
+          var args = [];
+          for (var i in sig) {
+            args.push(sig[i].split(' ').pop());
+          }
+          var func = `(${args}) => ${body};`;
+#if DYLINK_DEBUG
+          dbg(`adding new EM_JS function: ${name} = ${func}`);
+#endif
+          {{{ makeEval('wasmImports[name] = eval(func)') }}};
+        }
+
+        for (var name in moduleExports) {
+          if (name.startsWith('__em_js__')) {
+            var start = moduleExports[name]
+            {{{ from64('start') }}}
+            var jsString = UTF8ToString(start);
+            var parts = jsString.split('<::>');
+            addEmJs(name.replace('__em_js__', ''), parts[0], parts[1]);
+          }
+        }
 #endif
 
         // initialize the module

test/other/metadce/test_metadce_hello_dylink.jssize

@@ -1 +1 @@
-14888
+15211

test/other/test_em_js_main.c

@@ -0,0 +1,9 @@
+#include <stdio.h>
+#include <emscripten.h>
+
+int test_side();
+
+int main() {
+  printf("in main\n");
+  return test_side();
+}

test/other/test_em_js_main.out

@@ -0,0 +1,2 @@
+in main
+hello from side module 42 + hello

test/other/test_em_js_side.c

@@ -0,0 +1,11 @@
+#include <emscripten/em_js.h>
+#include <stdio.h>
+
+EM_JS(void*, js_side_func, (int num, char* ptr), {
+  out(`hello from side module ${num} + ${UTF8ToString(ptr)}`);
+  return 99;
+});
+
+void test_side() {
+  js_side_func(42, "hello");
+}

test/test_other.py

@@ -11964,8 +11964,8 @@ def test_runtime_keepalive(self):
 
   @crossplatform
   def test_em_js_side_module(self):
-    err = self.expect_fail([EMXX, '-sSIDE_MODULE', test_file('core/test_em_js.cpp')])
-    self.assertContained('EM_JS is not supported in side modules', err)
+    self.build(test_file('other/test_em_js_side.c'), js_outfile=False, emcc_args=['-sSIDE_MODULE'], output_basename='side')
+    self.do_other_test('test_em_js_main.c', emcc_args=['-sMAIN_MODULE=2', 'side.wasm'])
 
   def test_em_js_main_module(self):
     self.set_setting('MAIN_MODULE', 2)

tools/utils.py

@@ -29,6 +29,13 @@ def safe_ensure_dirs(dirname):
   os.makedirs(dirname, exist_ok=True)
 
 
+# TODO(sbc): Replace with str.removeprefix once we update to python3.9
+def removeprefix(string, prefix):
+  if string.startswith(prefix):
+    return string[len(prefix):]
+  return string
+
+
 @contextlib.contextmanager
 def chdir(dir):
   """A context manager that performs actions in the given directory."""