Centralize internal pthread exports. NFC (#16853)

The change moves from special case handling in `src/modules.js` to
`emcc.py` where other similar symbols are handled already.

Author: sbc100

emcc.py

@@ -1536,9 +1536,10 @@ def include_and_export(name):
 
   include_and_export('establishStackSpace')
   include_and_export('invokeEntryPoint')
+  include_and_export('PThread')
   if not settings.MINIMAL_RUNTIME:
     # keepRuntimeAlive does not apply to MINIMAL_RUNTIME.
-    settings.EXPORTED_RUNTIME_METHODS += ['keepRuntimeAlive']
+    settings.EXPORTED_RUNTIME_METHODS += ['keepRuntimeAlive', 'ExitStatus', 'wasmMemory']
 
   if settings.MODULARIZE:
     if not settings.EXPORT_ES6 and settings.EXPORT_NAME == 'Module':

src/modules.js

@@ -432,6 +432,7 @@ function exportRuntime() {
     'callMain',
     'abort',
     'keepRuntimeAlive',
+    'wasmMemory',
   ];
 
   if (USE_PTHREADS && ALLOW_MEMORY_GROWTH) {
@@ -479,6 +480,7 @@ function exportRuntime() {
       'lengthBytesUTF32',
       'allocateUTF8',
       'allocateUTF8OnStack',
+      'ExitStatus',
     ]);
   }
 
@@ -487,21 +489,6 @@ function exportRuntime() {
     runtimeElements.push('checkStackCookie');
   }
 
-  if (USE_PTHREADS) {
-    // In pthreads mode, the following functions always need to be exported to
-    // Module for closure compiler, and also for MODULARIZE (so worker.js can
-    // access them).
-    const threadExports = ['PThread', 'wasmMemory'];
-    if (!MINIMAL_RUNTIME) {
-      threadExports.push('ExitStatus');
-    }
-
-    threadExports.forEach((x) => {
-      EXPORTED_RUNTIME_METHODS_SET.add(x);
-      runtimeElements.push(x);
-    });
-  }
-
   if (SUPPORT_BASE64_EMBEDDING) {
     runtimeElements.push('intArrayFromBase64');
     runtimeElements.push('tryParseAsDataURI');