Skip to content

Commit b3db6a9

Browse files
committed
review feedback [ci skip]
1 parent 06ab316 commit b3db6a9

File tree

2 files changed

+1
-11
lines changed

2 files changed

+1
-11
lines changed

system/lib/compiler-rt/emscripten_exception_builtins.c

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,6 @@
88
/*
99
References to these longjmp- and exceptions-supporting things are generated
1010
in the llvm backend.
11-
12-
Note that these might make more sense in compiler-rt, but they need to be
13-
built with multithreading support when relevant (to avoid races between
14-
threads that throw at the same time etc.), and compiler-rt is not built
15-
that way atm.
1611
*/
1712

1813
#include <threads.h>

tests/core/pthread/exceptions.cpp

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -44,13 +44,8 @@ pthread_t thread[NUM_THREADS];
4444

4545
void CreateThread(int i)
4646
{
47-
pthread_attr_t attr;
48-
pthread_attr_init(&attr);
49-
pthread_attr_setdetachstate(&attr, PTHREAD_CREATE_JOINABLE);
50-
static int counter = 1;
51-
int rc = pthread_create(&thread[i], &attr, ThreadMain, (void*)i);
47+
int rc = pthread_create(&thread[i], nullptr, ThreadMain, (void*)i);
5248
assert(rc == 0);
53-
pthread_attr_destroy(&attr);
5449
}
5550

5651
void loop() {

0 commit comments

Comments
 (0)