Skip to content

Commit d5e39b7

Browse files
committed
[emmalloc] Fix verbose output
`toString()` is undeclared. Fixes a regression introduced in commit 04a0cad.
1 parent 58889f9 commit d5e39b7

File tree

1 file changed

+16
-16
lines changed

1 file changed

+16
-16
lines changed

system/lib/emmalloc.c

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -358,16 +358,16 @@ static void dump_memory_regions() {
358358
Region *r = (Region*)root;
359359
assert(debug_region_is_consistent(r));
360360
uint8_t *lastRegionEnd = root->endPtr;
361-
MAIN_THREAD_ASYNC_EM_ASM(out('Region block '+ptrToString($0)+' - '+ptrToString($1)+ ' ('+toString(Number($2))+' bytes):'),
361+
MAIN_THREAD_ASYNC_EM_ASM(out('Region block '+ptrToString($0)+' - '+ptrToString($1)+ ' ('+Number($2)+' bytes):'),
362362
r, lastRegionEnd, lastRegionEnd-(uint8_t*)r);
363363
while ((uint8_t*)r < lastRegionEnd) {
364-
MAIN_THREAD_ASYNC_EM_ASM(out('Region '+ptrToString($0)+', size: '+toString(Number($1))+' ('+($2?"used":"--FREE--")+')'),
364+
MAIN_THREAD_ASYNC_EM_ASM(out('Region '+ptrToString($0)+', size: '+Number($1)+' ('+($2?"used":"--FREE--")+')'),
365365
r, r->size, region_ceiling_size(r) == r->size);
366366

367367
assert(debug_region_is_consistent(r));
368368
size_t sizeFromCeiling = size_of_region_from_ceiling(r);
369369
if (sizeFromCeiling != r->size) {
370-
MAIN_THREAD_ASYNC_EM_ASM(out('Corrupt region! Size marker at the end of the region does not match: '+toString(Number($0))), sizeFromCeiling);
370+
MAIN_THREAD_ASYNC_EM_ASM(out('Corrupt region! Size marker at the end of the region does not match: '+Number($0)), sizeFromCeiling);
371371
}
372372
if (r->size == 0) {
373373
break;
@@ -382,7 +382,7 @@ static void dump_memory_regions() {
382382
Region *prev = &freeRegionBuckets[i];
383383
Region *fr = freeRegionBuckets[i].next;
384384
while (fr != &freeRegionBuckets[i]) {
385-
MAIN_THREAD_ASYNC_EM_ASM(out('In bucket '+$0+', free region '+ptrToString($1)+', size: ' + toString(Number($2)) + ' (size at ceiling: '+toString(Number($3))+'), prev: ' + ptrToString($4) + ', next: ' + ptrToString($5)),
385+
MAIN_THREAD_ASYNC_EM_ASM(out('In bucket '+$0+', free region '+ptrToString($1)+', size: ' + Number($2) + ' (size at ceiling: '+Number($3)+'), prev: ' + ptrToString($4) + ', next: ' + ptrToString($5)),
386386
i, fr, fr->size, size_of_region_from_ceiling(fr), fr->prev, fr->next);
387387
assert(debug_region_is_consistent(fr));
388388
assert(region_is_free(fr));
@@ -393,7 +393,7 @@ static void dump_memory_regions() {
393393
fr = fr->next;
394394
}
395395
}
396-
MAIN_THREAD_ASYNC_EM_ASM(out('Free bucket index map: ' + toString(Number($0)).toString(2) + ' ' + toString(Number($1)).toString(2)), (uint32_t)(freeRegionBucketsUsed >> 32), (uint32_t)freeRegionBucketsUsed);
396+
MAIN_THREAD_ASYNC_EM_ASM(out('Free bucket index map: ' + Number($0).toString(2) + ' ' + Number($1).toString(2)), (uint32_t)(freeRegionBucketsUsed >> 32), (uint32_t)freeRegionBucketsUsed);
397397
MAIN_THREAD_ASYNC_EM_ASM(out(""));
398398
}
399399

@@ -409,14 +409,14 @@ static int validate_memory_regions() {
409409
while (root) {
410410
Region *r = (Region*)root;
411411
if (!debug_region_is_consistent(r)) {
412-
MAIN_THREAD_ASYNC_EM_ASM(err('Used region '+ptrToString($0)+', size: '+toString(Number($1))+' ('+($2?"used":"--FREE--")+') is corrupt (size markers in the beginning and at the end of the region do not match!)'),
412+
MAIN_THREAD_ASYNC_EM_ASM(err('Used region '+ptrToString($0)+', size: '+Number($1)+' ('+($2?"used":"--FREE--")+') is corrupt (size markers in the beginning and at the end of the region do not match!)'),
413413
r, r->size, region_ceiling_size(r) == r->size);
414414
return 1;
415415
}
416416
uint8_t *lastRegionEnd = root->endPtr;
417417
while ((uint8_t*)r < lastRegionEnd) {
418418
if (!debug_region_is_consistent(r)) {
419-
MAIN_THREAD_ASYNC_EM_ASM(err('Used region '+ptrToString($0)+', size: '+toString(Number($1))+' ('+($2?"used":"--FREE--")+') is corrupt (size markers in the beginning and at the end of the region do not match!)'),
419+
MAIN_THREAD_ASYNC_EM_ASM(err('Used region '+ptrToString($0)+', size: '+Number($1)+' ('+($2?"used":"--FREE--")+') is corrupt (size markers in the beginning and at the end of the region do not match!)'),
420420
r, r->size, region_ceiling_size(r) == r->size);
421421
return 1;
422422
}
@@ -432,7 +432,7 @@ static int validate_memory_regions() {
432432
Region *fr = freeRegionBuckets[i].next;
433433
while (fr != &freeRegionBuckets[i]) {
434434
if (!debug_region_is_consistent(fr) || !region_is_free(fr) || fr->prev != prev || fr->next == fr || fr->prev == fr) {
435-
MAIN_THREAD_ASYNC_EM_ASM(out('In bucket '+$0+', free region '+ptrToString($1)+', size: ' + toString(Number($2)) + ' (size at ceiling: '+toString(Number($3))+'), prev: ' + ptrToString($4) + ', next: 0x' + ptrToString($5) + ' is corrupt!'),
435+
MAIN_THREAD_ASYNC_EM_ASM(out('In bucket '+$0+', free region '+ptrToString($1)+', size: ' + Number($2) + ' (size at ceiling: '+Number($3)+'), prev: ' + ptrToString($4) + ', next: 0x' + ptrToString($5) + ' is corrupt!'),
436436
i, fr, fr->size, size_of_region_from_ceiling(fr), fr->prev, fr->next);
437437
return 1;
438438
}
@@ -629,7 +629,7 @@ static void *attempt_allocate(Region *freeRegion, size_t alignment, size_t size)
629629
#endif
630630

631631
#ifdef EMMALLOC_VERBOSE
632-
MAIN_THREAD_ASYNC_EM_ASM(out('attempt_allocate - succeeded allocating memory, region ptr=' + ptrToString($0) + ', align=' + $1 + ', payload size=' + toString(Number($2)) + ' bytes)'), freeRegion, alignment, size);
632+
MAIN_THREAD_ASYNC_EM_ASM(out('attempt_allocate - succeeded allocating memory, region ptr=' + ptrToString($0) + ', align=' + $1 + ', payload size=' + Number($2) + ' bytes)'), freeRegion, alignment, size);
633633
#endif
634634

635635
return (uint8_t*)freeRegion + sizeof(size_t);
@@ -659,7 +659,7 @@ static void *allocate_memory(size_t alignment, size_t size) {
659659
ASSERT_MALLOC_IS_ACQUIRED();
660660

661661
#ifdef EMMALLOC_VERBOSE
662-
MAIN_THREAD_ASYNC_EM_ASM(out('allocate_memory(align=' + $0 + ', size=' + toString(Number($1)) + ' bytes)'), alignment, size);
662+
MAIN_THREAD_ASYNC_EM_ASM(out('allocate_memory(align=' + $0 + ', size=' + Number($1) + ' bytes)'), alignment, size);
663663
#endif
664664

665665
#ifdef EMMALLOC_MEMVALIDATE
@@ -675,7 +675,7 @@ static void *allocate_memory(size_t alignment, size_t size) {
675675

676676
if (size > MAX_ALLOC_SIZE) {
677677
#ifdef EMMALLOC_VERBOSE
678-
MAIN_THREAD_ASYNC_EM_ASM(out('Allocation failed: attempted allocation size is too large: ' + toString(Number($0)) + 'bytes! (negative integer wraparound?)'), size);
678+
MAIN_THREAD_ASYNC_EM_ASM(out('Allocation failed: attempted allocation size is too large: ' + Number($0) + 'bytes! (negative integer wraparound?)'), size);
679679
#endif
680680
return 0;
681681
}
@@ -930,7 +930,7 @@ static int attempt_region_resize(Region *region, size_t size) {
930930
assert(HAS_ALIGNMENT(size, sizeof(size_t)));
931931

932932
#ifdef EMMALLOC_VERBOSE
933-
MAIN_THREAD_ASYNC_EM_ASM(out('attempt_region_resize(region=' + ptrToString($0) + ', size=' + toString(Number($1)) + ' bytes)'), region, size);
933+
MAIN_THREAD_ASYNC_EM_ASM(out('attempt_region_resize(region=' + ptrToString($0) + ', size=' + Number($1) + ' bytes)'), region, size);
934934
#endif
935935

936936
// First attempt to resize this region, if the next region that follows this one
@@ -988,7 +988,7 @@ static int acquire_and_attempt_region_resize(Region *region, size_t size) {
988988

989989
void *emmalloc_aligned_realloc(void *ptr, size_t alignment, size_t size) {
990990
#ifdef EMMALLOC_VERBOSE
991-
MAIN_THREAD_ASYNC_EM_ASM(out('aligned_realloc(ptr=' + ptrToString($0) + ', alignment=' + $1 + ', size=' + toString(Number($2))), ptr, alignment, size);
991+
MAIN_THREAD_ASYNC_EM_ASM(out('aligned_realloc(ptr=' + ptrToString($0) + ', alignment=' + $1 + ', size=' + Number($2)), ptr, alignment, size);
992992
#endif
993993

994994
if (!ptr) {
@@ -1002,7 +1002,7 @@ void *emmalloc_aligned_realloc(void *ptr, size_t alignment, size_t size) {
10021002

10031003
if (size > MAX_ALLOC_SIZE) {
10041004
#ifdef EMMALLOC_VERBOSE
1005-
MAIN_THREAD_ASYNC_EM_ASM(out('Allocation failed: attempted allocation size is too large: ' + toString(Number($0)) + 'bytes! (negative integer wraparound?)'), size);
1005+
MAIN_THREAD_ASYNC_EM_ASM(out('Allocation failed: attempted allocation size is too large: ' + Number($0) + 'bytes! (negative integer wraparound?)'), size);
10061006
#endif
10071007
return 0;
10081008
}
@@ -1052,7 +1052,7 @@ void *emmalloc_realloc_try(void *ptr, size_t size) {
10521052

10531053
if (size > MAX_ALLOC_SIZE) {
10541054
#ifdef EMMALLOC_VERBOSE
1055-
MAIN_THREAD_ASYNC_EM_ASM(out('Allocation failed: attempted allocation size is too large: ' + toString(Number($0)) + 'bytes! (negative integer wraparound?)'), size);
1055+
MAIN_THREAD_ASYNC_EM_ASM(out('Allocation failed: attempted allocation size is too large: ' + Number($0) + 'bytes! (negative integer wraparound?)'), size);
10561056
#endif
10571057
return 0;
10581058
}
@@ -1086,7 +1086,7 @@ void *emmalloc_aligned_realloc_uninitialized(void *ptr, size_t alignment, size_t
10861086

10871087
if (size > MAX_ALLOC_SIZE) {
10881088
#ifdef EMMALLOC_VERBOSE
1089-
MAIN_THREAD_ASYNC_EM_ASM(out('Allocation failed: attempted allocation size is too large: ' + toString(Number($0)) + 'bytes! (negative integer wraparound?)'), size);
1089+
MAIN_THREAD_ASYNC_EM_ASM(out('Allocation failed: attempted allocation size is too large: ' + Number($0) + 'bytes! (negative integer wraparound?)'), size);
10901090
#endif
10911091
return 0;
10921092
}

0 commit comments

Comments
 (0)