Alon Zakai

@MikeHolman 6 months sounds like a long time, so I guess we should look into function splitting on the toolchain side even if godot has a manual workaround - thoughts?...

Yes, sorry about that, it inherits from the suite so it prints the help there. To fix this we'd need to refactor `scripts/test/shared.py` which atm parses arguments at the top...

This may be the same issue as #6989, as Asyncify uses Flatten internally. If not, please upload the file as an archive and I can take a look.

Thanks. This is a real Asyncify issue, it turns out. Reduced testcase: ```wat (module (import "a" "b" (func $import)) (func $0 (result (ref i31)) (call $import) (unreachable) ) ) ```...

Interesting. I think what's going on here is that the dynamic linking conventions specify that that section needs to be first: https://github.com/WebAssembly/tool-conventions/blob/master/DynamicLinking.md#the-dylink-section and emscripten emits it first, but none of...

Heh, yeah, great minds and all that... The debuggability is really nice, so if you can't measure a regression here, it's probably worthwhile?

Does this not affect e.g. `./emcc test/hello_world.c -lxlib.js`? That works now but after this change it would need to be `./emcc test/hello_world.c -lX11.js`

> To fix the type problem, we should keep our string type but make it a subtype of extern rather than any Makes sense, I see how making string a...

Great, thanks, that's what I was missing then. Good to know. I'll look into making stringref a subtype of externref then, to start.

Fuzzed heavily over the weekend for any diff vs the previous behavior, no issues found.