Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(841)

Unified Diff: runtime/vm/intrinsifier_arm.cc

Issue 1263513002: VM: Load allocation-top and -end via Thread. (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: fixed cc tests Created 5 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: runtime/vm/intrinsifier_arm.cc
diff --git a/runtime/vm/intrinsifier_arm.cc b/runtime/vm/intrinsifier_arm.cc
index 702ed7f859bd8fbd44e56a1a8179cdfab78d5fc2..70ce88346e526fa989d24e47d605ded25a2a66f9 100644
--- a/runtime/vm/intrinsifier_arm.cc
+++ b/runtime/vm/intrinsifier_arm.cc
@@ -189,7 +189,8 @@ void Intrinsifier::GrowableArray_add(Assembler* assembler) {
#define TYPED_ARRAY_ALLOCATION(type_name, cid, max_len, scale_shift) \
Label fall_through; \
const intptr_t kArrayLengthStackOffset = 0 * kWordSize; \
- __ MaybeTraceAllocation(cid, R2, &fall_through); \
+ __ MaybeTraceAllocation(cid, R2, &fall_through, \
+ /* inline_isolate = */ false); \
__ ldr(R2, Address(SP, kArrayLengthStackOffset)); /* Array length. */ \
/* Check that length is a positive Smi. */ \
/* R2: requested array length argument. */ \
@@ -206,10 +207,9 @@ void Intrinsifier::GrowableArray_add(Assembler* assembler) {
const intptr_t fixed_size = sizeof(Raw##type_name) + kObjectAlignment - 1; \
__ AddImmediate(R2, fixed_size); \
__ bic(R2, R2, Operand(kObjectAlignment - 1)); \
- Heap* heap = Isolate::Current()->heap(); \
- Heap::Space space = heap->SpaceForAllocation(cid); \
- __ LoadImmediate(R0, heap->TopAddress(space)); \
- __ ldr(R0, Address(R0, 0)); \
+ Heap::Space space = Heap::SpaceForAllocation(cid); \
+ __ ldr(R3, Address(THR, Thread::heap_offset())); \
+ __ ldr(R0, Address(R3, Heap::TopOffset(space))); \
\
/* R2: allocation size. */ \
__ adds(R1, R0, Operand(R2)); \
@@ -219,16 +219,15 @@ void Intrinsifier::GrowableArray_add(Assembler* assembler) {
/* R0: potential new object start. */ \
/* R1: potential next object start. */ \
/* R2: allocation size. */ \
- __ LoadImmediate(R3, heap->EndAddress(space)); \
- __ ldr(R3, Address(R3, 0)); \
- __ cmp(R1, Operand(R3)); \
+ /* R3: heap. */ \
+ __ ldr(IP, Address(R3, Heap::EndOffset(space))); \
+ __ cmp(R1, Operand(IP)); \
__ b(&fall_through, CS); \
\
/* Successfully allocated the object(s), now update top to point to */ \
/* next object start and initialize the object. */ \
- __ LoadAllocationStatsAddress(R4, cid); \
- __ LoadImmediate(R3, heap->TopAddress(space)); \
- __ str(R1, Address(R3, 0)); \
+ __ LoadAllocationStatsAddress(R4, cid, /* inline_isolate = */ false); \
+ __ str(R1, Address(R3, Heap::TopOffset(space))); \
__ AddImmediate(R0, kHeapObjectTag); \
/* Initialize the tags. */ \
/* R0: new object start as a tagged pointer. */ \
@@ -1781,12 +1780,10 @@ static void TryAllocateOnebyteString(Assembler* assembler,
__ AddImmediate(length_reg, fixed_size);
__ bic(length_reg, length_reg, Operand(kObjectAlignment - 1));
- Isolate* isolate = Isolate::Current();
- Heap* heap = isolate->heap();
const intptr_t cid = kOneByteStringCid;
- Heap::Space space = heap->SpaceForAllocation(cid);
- __ LoadImmediate(R3, heap->TopAddress(space));
- __ ldr(R0, Address(R3, 0));
+ Heap::Space space = Heap::SpaceForAllocation(cid);
+ __ ldr(R3, Address(THR, Thread::heap_offset()));
+ __ ldr(R0, Address(R3, Heap::TopOffset(space)));
// length_reg: allocation size.
__ adds(R1, R0, Operand(length_reg));
@@ -1796,16 +1793,15 @@ static void TryAllocateOnebyteString(Assembler* assembler,
// R0: potential new object start.
// R1: potential next object start.
// R2: allocation size.
- // R3: heap->TopAddress(space).
- __ LoadImmediate(R7, heap->EndAddress(space));
- __ ldr(R7, Address(R7, 0));
+ // R3: heap.
+ __ ldr(R7, Address(R3, Heap::EndOffset(space)));
__ cmp(R1, Operand(R7));
__ b(&fail, CS);
// Successfully allocated the object(s), now update top to point to
// next object start and initialize the object.
- __ LoadAllocationStatsAddress(R4, cid);
- __ str(R1, Address(R3, 0));
+ __ LoadAllocationStatsAddress(R4, cid, /* inline_isolate = */ false);
+ __ str(R1, Address(R3, Heap::TopOffset(space)));
__ AddImmediate(R0, kHeapObjectTag);
// Initialize the tags.

Powered by Google App Engine
This is Rietveld 408576698