Remove __pthread_detached_exit in favor of __emscripten_thread_cleanup. NFC.

Author: kleisauke

src/library_pthread.js

@@ -275,11 +275,6 @@ var LibraryPThread = {
           err('Thread ' + d['threadId'] + ': ' + d['text']);
         } else if (cmd === 'alert') {
           alert('Thread ' + d['threadId'] + ': ' + d['text']);
-        } else if (cmd === 'detachedExit') {
-#if ASSERTIONS
-          assert(worker.pthread);
-#endif
-          PThread.returnWorkerToPool(worker);
         } else if (d.target === 'setimmediate') {
           // Worker wants to postMessage() to itself to implement setImmediate()
           // emulation.
@@ -307,7 +302,7 @@ var LibraryPThread = {
         worker.on('error', function(e) {
           worker.onerror(e);
         });
-        worker.on('detachedExit', function() {
+        worker.on('exit', function() {
           // TODO: update the worker queue?
           // See: https://github.com/emscripten-core/emscripten/issues/9763
         });
@@ -816,12 +811,6 @@ var LibraryPThread = {
     return 0;
   },
 
-  __pthread_detached_exit: function() {
-    // Called at the end of pthread_exit (which occurs also when leaving the
-    // thread main function) if an only if the thread is in a detached state.
-    postMessage({ 'cmd': 'detachedExit' });
-  },
-
   // Returns 0 on success, or one of the values -ETIMEDOUT, -EWOULDBLOCK or -EINVAL on error.
   emscripten_futex_wait__deps: ['emscripten_main_thread_process_queued_calls'],
   emscripten_futex_wait: function(addr, val, timeout) {

system/lib/pthread/pthread_create.c

@@ -25,7 +25,7 @@
 extern int __pthread_create_js(pthread_t thread, const pthread_attr_t *attr, void *(*start_routine) (void *), void *arg);
 extern void _emscripten_thread_init(int, int, int);
 extern int _emscripten_default_pthread_stack_size();
-extern void __pthread_detached_exit();
+extern void __emscripten_thread_cleanup(pthread_t thread);
 extern void* _emscripten_tls_base();
 extern int8_t __dso_handle;
 
@@ -236,7 +236,7 @@ void _emscripten_thread_exit(void* result) {
   // object and we are done.
   if (state == DT_DETACHED) {
     self->detach_state = DT_EXITED;
-    __pthread_detached_exit();
+    __emscripten_thread_cleanup(self);
   } else {
     self->detach_state = DT_EXITING;
     // wake any threads that might be waiting for us to exit