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

Unified Diff: runtime/vm/stub_code_x64.cc

Issue 2112043002: Land Ivan's change of 'Remove support for verified memory handling' (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Address code review comments. Created 4 years, 6 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
« no previous file with comments | « runtime/vm/stub_code_ia32.cc ('k') | runtime/vm/verified_memory.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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).
« no previous file with comments | « runtime/vm/stub_code_ia32.cc ('k') | runtime/vm/verified_memory.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698