[JSPI] Pause / resume main loop before executing an async functions (matching behavior of ASYNCIFY=1)

[inolen]

Seems straight forward, but would appreciate someone more familiar with this code sanity checking.

[sbc100]

Seems reasonable yes. Maybe we can write a test for this the runs in both modes?

[sbc100]

I wonder if we should put these 2 lines in the finally block? Then we don't need t new res variable I think.

[inolen]

Added a JSPI specific test, I'm not quite sure the best way to have that run for both modes.

Hate adding tests with sleeps, but not sure of a better way to test that here.

[sbc100]

I think you can use @with_asyncify_and_jspi here to get both modes.

Then maybe the test itself should be called test_asyncify_mainloop?

[sbc100]

No space inside braces.

[inolen]

Just realized this is failing because of a change I didn't commit, but I'm not sure if you want this right now -

diff --git a/src/lib/libeventloop.js b/src/lib/libeventloop.js
index ad59b9262..3646198a2 100644
--- a/src/lib/libeventloop.js
+++ b/src/lib/libeventloop.js
@@ -378,6 +378,9 @@ LibraryJSEventLoop = {
   emscripten_set_main_loop__deps: ['$setMainLoop'],
   emscripten_set_main_loop: (func, fps, simulateInfiniteLoop) => {
     var iterFunc = {{{ makeDynCall('v', 'func') }}};
+#if JSPI
+    iterFunc = WebAssembly.promising(iterFunc);
+#endif
     setMainLoop(iterFunc, fps, simulateInfiniteLoop);
   },

I believe I had read some chatter here somewhere about automating this vs sprinkling it around everywhere.

[sbc100]

Just realized this is failing because of a change I didn't commit, but I'm not sure if you want this right now -

diff --git a/src/lib/libeventloop.js b/src/lib/libeventloop.js
index ad59b9262..3646198a2 100644
--- a/src/lib/libeventloop.js
+++ b/src/lib/libeventloop.js
@@ -378,6 +378,9 @@ LibraryJSEventLoop = {
   emscripten_set_main_loop__deps: ['$setMainLoop'],
   emscripten_set_main_loop: (func, fps, simulateInfiniteLoop) => {
     var iterFunc = {{{ makeDynCall('v', 'func') }}};
+#if JSPI
+    iterFunc = WebAssembly.promising(iterFunc);
+#endif
     setMainLoop(iterFunc, fps, simulateInfiniteLoop);
   },

I believe I had read some chatter here somewhere about automating this vs sprinkling it around everywhere.

Isn't the solution here to make sure the interFunc is in JSPI_EXPORTS? @brendandahl

[inolen]

Hmm, I tried adding a '-sJSPI_EXPORTS=loop' into the cflags variable there, but I can't get that function to show up in the exports array passed to instrumentWasmExports.

Ah, that finally worked if I also added '-sEXPORTED_FUNCTIONS=_main,_loop'.