Skip to content

Commit dd5430a

Browse files
committed
simplify more
1 parent 979a0b8 commit dd5430a

File tree

1 file changed

+27
-22
lines changed

1 file changed

+27
-22
lines changed

std/assembly/internal/runtime.ts

Lines changed: 27 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -1,22 +1,28 @@
11
import { AL_MASK } from "./allocator";
22

3-
/** Common runtime header used by all objects. */
3+
/** Common runtime header of all objects. */
44
@unmanaged
55
export class HEADER {
6-
/** Unique id of the respective class. Not yet registered with GC if zero.*/
6+
/** Unique id of the respective class or a magic value if not yet registered.*/
77
classId: u32;
88
/** Size of the allocated payload. */
99
payloadSize: u32;
10-
/** Reserved field for use by GC. */
10+
/** Reserved field for use by GC. Only present if GC is. */
1111
reserved1: usize; // itcm: tagged next
12-
/** Reserved field for use by GC. */
12+
/** Reserved field for use by GC. Only present if GC is. */
1313
reserved2: usize; // itcm: prev
1414
}
1515

16+
/** Whether a GC is present or not. */
17+
@inline export const GC = true;
18+
1619
/** Size of the common runtime header. */
17-
@inline export const HEADER_SIZE: usize = (offsetof<HEADER>() + AL_MASK) & ~AL_MASK;
18-
/** Magic value used to validate common headers. */
19-
@inline export const HEADER_MAGIC: usize = <usize>0xA55E4B17;
20+
@inline export const HEADER_SIZE: usize = GC
21+
? (offsetof<HEADER>( ) + AL_MASK) & ~AL_MASK // full header if GC is present
22+
: (offsetof<HEADER>("reserved1") + AL_MASK) & ~AL_MASK; // half header if GC is absent
23+
24+
/** Magic value used to validate common runtime headers. */
25+
@inline export const HEADER_MAGIC: u32 = 0xA55E4B17;
2026

2127
/** Aligns an allocation to actual block size. */
2228
function ALIGN(payloadSize: usize): usize {
@@ -33,19 +39,19 @@ function ALIGN(payloadSize: usize): usize {
3339
function UNREF(ref: usize): HEADER {
3440
assert(ref >= HEAP_BASE + HEADER_SIZE); // must be a heap object
3541
var header = changetype<HEADER>(ref - HEADER_SIZE);
36-
assert(!header.classId && header.reserved2 == HEADER_MAGIC); // must be unregistered
42+
assert(header.classId == HEADER_MAGIC); // must be unregistered
3743
return header;
3844
}
3945

40-
// === General allocation/deallocation ============================================================
41-
4246
/** Allocates a new object and returns a pointer to its payload. */
4347
export function ALLOC(payloadSize: u32): usize {
4448
var header = changetype<HEADER>(memory.allocate(ALIGN(payloadSize)));
45-
header.classId = 0;
49+
header.classId = HEADER_MAGIC;
4650
header.payloadSize = payloadSize;
47-
header.reserved1 = 0;
48-
header.reserved2 = HEADER_MAGIC;
51+
if (GC) {
52+
header.reserved1 = 0;
53+
header.reserved2 = 0;
54+
}
4955
var ref = changetype<usize>(header) + HEADER_SIZE;
5056
memory.fill(ref, 0, payloadSize);
5157
return ref;
@@ -60,9 +66,11 @@ export function REALLOC(ref: usize, newPayloadSize: u32): usize {
6066
if (ALIGN(payloadSize) < newAlignedSize) {
6167
// move if the allocation isn't large enough to hold the new payload
6268
let newHeader = changetype<HEADER>(memory.allocate(newAlignedSize));
63-
newHeader.classId = 0;
64-
newHeader.reserved1 = 0;
65-
newHeader.reserved2 = HEADER_MAGIC;
69+
newHeader.classId = HEADER_MAGIC;
70+
if (GC) {
71+
newHeader.reserved1 = 0;
72+
newHeader.reserved2 = 0;
73+
}
6674
let newRef = changetype<usize>(newHeader) + HEADER_SIZE;
6775
memory.copy(newRef, ref, payloadSize);
6876
memory.fill(newRef + payloadSize, 0, newPayloadSize - payloadSize);
@@ -91,22 +99,19 @@ export function FREE(ref: usize): void {
9199
/** Registers a managed object with GC. */
92100
export function REGISTER<T>(ref: usize, parentRef: usize): void {
93101
var header = UNREF(ref);
94-
header.classId = /* TODO: CLASSID<T>() */ 1;
95-
header.reserved2 = 0;
102+
header.classId = /* TODO: CLASSID<T>() */ 1;
96103
// TODO
97104
}
98105

99-
// === ArrayBuffer ================================================================================
100-
106+
/** ArrayBuffer base class. */
101107
export abstract class ArrayBufferBase {
102108
get byteLength(): i32 {
103109
var header = changetype<HEADER>(changetype<usize>(this) - HEADER_SIZE);
104110
return header.payloadSize;
105111
}
106112
}
107113

108-
// === String =====================================================================================
109-
114+
/** String base class. */
110115
export abstract class StringBase {
111116
get length(): i32 {
112117
var header = changetype<HEADER>(changetype<usize>(this) - HEADER_SIZE);

0 commit comments

Comments
 (0)