Skip to content

Commit 464a3da

Browse files
committed
simplify
1 parent cfd2748 commit 464a3da

File tree

7 files changed

+28
-34
lines changed

7 files changed

+28
-34
lines changed

system/lib/sbrk.c

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -29,18 +29,13 @@
2929
#define SET_ERRNO()
3030
#endif
3131

32-
static intptr_t sbrk_val = 0;
33-
static intptr_t* sbrk_ptr = &sbrk_val;
34-
3532
extern size_t __heap_base;
3633

34+
static intptr_t sbrk_val = (intptr_t)&__heap_base;
35+
static intptr_t* sbrk_ptr = &sbrk_val;
36+
3737
intptr_t* emscripten_get_sbrk_ptr() {
38-
if (sbrk_val == 0) {
39-
sbrk_val = (intptr_t)&__heap_base;
40-
sbrk_val += 1024; // FIXME FIXME remove this once no JS static allocs
41-
sbrk_val = (sbrk_val + 3) & ~3;
42-
}
43-
return &sbrk_val;
38+
return sbrk_ptr;
4439
}
4540

4641
void *sbrk(intptr_t increment) {
@@ -55,7 +50,6 @@ void *sbrk(intptr_t increment) {
5550
intptr_t expected;
5651
while (1) {
5752
#endif // __EMSCRIPTEN_PTHREADS__
58-
intptr_t* sbrk_ptr = emscripten_get_sbrk_ptr(); // TODO remove with above hack
5953
#if __EMSCRIPTEN_PTHREADS__
6054
intptr_t old_brk = __c11_atomic_load((_Atomic(intptr_t)*)sbrk_ptr, __ATOMIC_SEQ_CST);
6155
#else

tests/code_size/hello_webgl2_wasm.json

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@
33
"a.html.gz": 377,
44
"a.js": 5044,
55
"a.js.gz": 2401,
6-
"a.wasm": 10937,
7-
"a.wasm.gz": 6947,
6+
"a.wasm": 10924,
7+
"a.wasm.gz": 6939,
88
"total": 16544,
9-
"total_gz": 9725
9+
"total_gz": 9717
1010
}
Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
{
22
"a.html": 588,
33
"a.html.gz": 386,
4-
"a.js": 22203,
5-
"a.js.gz": 8491,
6-
"a.mem": 3168,
7-
"a.mem.gz": 2711,
8-
"total": 25938,
9-
"total_gz": 11588
4+
"a.js": 22166,
5+
"a.js.gz": 8477,
6+
"a.mem": 3171,
7+
"a.mem.gz": 2715,
8+
"total": 25922,
9+
"total_gz": 11578
1010
}

tests/code_size/hello_webgl_wasm.json

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@
33
"a.html.gz": 377,
44
"a.js": 4529,
55
"a.js.gz": 2223,
6-
"a.wasm": 10937,
7-
"a.wasm.gz": 6947,
6+
"a.wasm": 10924,
7+
"a.wasm.gz": 6939,
88
"total": 16029,
9-
"total_gz": 9547
9+
"total_gz": 9539
1010
}
Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
{
22
"a.html": 588,
33
"a.html.gz": 386,
4-
"a.js": 21692,
5-
"a.js.gz": 8327,
6-
"a.mem": 3168,
7-
"a.mem.gz": 2711,
8-
"total": 25427,
9-
"total_gz": 11424
4+
"a.js": 21655,
5+
"a.js.gz": 8315,
6+
"a.mem": 3171,
7+
"a.mem.gz": 2715,
8+
"total": 25411,
9+
"total_gz": 11416
1010
}
Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"a.html": 13723,
3-
"a.html.gz": 7355,
4-
"total": 13727,
5-
"total_gz": 7355
3+
"a.html.gz": 7354,
4+
"total": 13723,
5+
"total_gz": 7354
66
}
Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"a.html": 19379,
3-
"a.html.gz": 8066,
4-
"total": 19387,
5-
"total_gz": 8066
3+
"a.html.gz": 8065,
4+
"total": 19379,
5+
"total_gz": 8065
66
}

0 commit comments

Comments
 (0)