Index: runtime/vm/stub_code_x64.cc |
diff --git a/runtime/vm/stub_code_x64.cc b/runtime/vm/stub_code_x64.cc |
index 25b5833fe9eae8e32e64d164670eea00fce6822e..4ea8f2b83e55ce2f6f64bbd41238753178408a2d 100644 |
--- a/runtime/vm/stub_code_x64.cc |
+++ b/runtime/vm/stub_code_x64.cc |
@@ -339,8 +339,8 @@ static void PushArgumentsArray(Assembler* assembler) { |
__ jmp(&loop_condition, kJumpLength); |
__ Bind(&loop); |
__ movq(RDI, Address(R12, 0)); |
- // No generational barrier needed, since array is in new space. |
- __ InitializeFieldNoBarrier(RAX, Address(RBX, 0), RDI); |
+ // Generational barrier is needed, array is not necessarily in new space. |
+ __ StoreIntoObject(RAX, Address(RBX, 0), RDI); |
__ addq(RBX, Immediate(kWordSize)); |
__ subq(R12, Immediate(kWordSize)); |
__ Bind(&loop_condition); |
@@ -641,12 +641,13 @@ void StubCode::GenerateAllocateArrayStub(Assembler* assembler) { |
// RAX: new object start as a tagged pointer. |
// Store the type argument field. |
- __ InitializeFieldNoBarrier(RAX, |
+ // No generetional barrier needed, since we store into a new object. |
+ __ StoreIntoObjectNoBarrier(RAX, |
FieldAddress(RAX, Array::type_arguments_offset()), |
RBX); |
// Set the length field. |
- __ InitializeFieldNoBarrier(RAX, |
+ __ StoreIntoObjectNoBarrier(RAX, |
FieldAddress(RAX, Array::length_offset()), |
R10); |
@@ -668,7 +669,7 @@ void StubCode::GenerateAllocateArrayStub(Assembler* assembler) { |
#endif // DEBUG |
__ j(ABOVE_EQUAL, &done, kJumpLength); |
// No generational barrier needed, since we are storing null. |
- __ InitializeFieldNoBarrier(RAX, Address(RDI, 0), R12); |
+ __ StoreIntoObjectNoBarrier(RAX, Address(RDI, 0), R12); |
__ addq(RDI, Immediate(kWordSize)); |
__ jmp(&init_loop, kJumpLength); |
__ Bind(&done); |
@@ -913,7 +914,7 @@ void StubCode::GenerateAllocateContextStub(Assembler* assembler) { |
// RAX: new object. |
// R10: number of context variables. |
// No generational barrier needed, since we are storing null. |
- __ InitializeFieldNoBarrier(RAX, |
+ __ StoreIntoObjectNoBarrier(RAX, |
FieldAddress(RAX, Context::parent_offset()), |
R9); |
@@ -932,7 +933,7 @@ void StubCode::GenerateAllocateContextStub(Assembler* assembler) { |
__ Bind(&loop); |
__ decq(R10); |
// No generational barrier needed, since we are storing null. |
- __ InitializeFieldNoBarrier(RAX, |
+ __ StoreIntoObjectNoBarrier(RAX, |
Address(R13, R10, TIMES_8, 0), |
R9); |
__ Bind(&entry); |
@@ -1093,7 +1094,7 @@ void StubCode::GenerateAllocationStubForClass(Assembler* assembler, |
for (intptr_t current_offset = Instance::NextFieldOffset(); |
current_offset < instance_size; |
current_offset += kWordSize) { |
- __ InitializeFieldNoBarrier(RAX, |
+ __ StoreIntoObjectNoBarrier(RAX, |
FieldAddress(RAX, current_offset), |
R9); |
} |
@@ -1114,16 +1115,17 @@ void StubCode::GenerateAllocationStubForClass(Assembler* assembler, |
static const bool kJumpLength = Assembler::kNearJump; |
#endif // DEBUG |
__ j(ABOVE_EQUAL, &done, kJumpLength); |
- __ InitializeFieldNoBarrier(RAX, Address(RCX, 0), R9); |
+ __ StoreIntoObjectNoBarrier(RAX, Address(RCX, 0), R9); |
__ addq(RCX, Immediate(kWordSize)); |
__ jmp(&init_loop, Assembler::kNearJump); |
__ Bind(&done); |
} |
if (is_cls_parameterized) { |
+ // RAX: new object (tagged). |
// RDX: new object type arguments. |
// Set the type arguments in the new object. |
intptr_t offset = cls.type_arguments_field_offset(); |
- __ InitializeFieldNoBarrier(RAX, FieldAddress(RAX, offset), RDX); |
+ __ StoreIntoObjectNoBarrier(RAX, FieldAddress(RAX, offset), RDX); |
} |
// Done allocating and initializing the instance. |
// RAX: new object (tagged). |