wasmExports not declared

[walkingeyerobot]

I'm getting an error about wasmExports not being declared. I believe this is due to #19901 combined with the odd set of linker options I have:

-s MODULARIZE=1
-s EXPORT_NAME=loadWasm
-s ENVIRONMENT=web,node
-s MINIMAL_RUNTIME=2
-s MINIMAL_RUNTIME_STREAMING_WASM_COMPILATION=1
-s FILESYSTEM=0
-s TEXTDECODER=2
-s POLYFILL=0

Oddly, this didn't happen before #19901 because I was running through the js compiler (I'm doing this myself manually and not via emscripten) and asm the compiler decided to declare and rename asm for me, but for some reason it won't do the same for wasmExports.

[sbc100]

Where is the reference to wasmExports coming from? Is it an internal emscripten reference, or is it being referenced perhaps in --pre-js or JS library code?

[walkingeyerobot]

It was erroring out in postamble-minimal.js. If I passed -lexports.js it worked in both cases because of these #ifs:
https://github.com/emscripten-core/emscripten/blob/main/src/postamble_minimal.js#L103-L105
https://github.com/emscripten-core/emscripten/blob/main/src/postamble_minimal.js#L143-L148

[sbc100]

What is the error you are seeing?

[juj]

Are you using embind in the project?

Can you post the line in the build that refers to wasmExports that fails? (and some lines of context around that to help locate the construct?)

[walkingeyerobot]

Yes, I'm using embind.

The line that fails is this: https://github.com/emscripten-core/emscripten/blob/main/src/postamble_minimal.js#L166

[sbc100]

Fix is in #20159