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

Unified Diff: runtime/vm/stub_code_mips.cc

Issue 62953014: ARM and MIPS changes which correspond to the initial change submitted in (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 7 years, 1 month 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_arm.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/stub_code_mips.cc
===================================================================
--- runtime/vm/stub_code_mips.cc (revision 30114)
+++ runtime/vm/stub_code_mips.cc (working copy)
@@ -39,10 +39,11 @@
const intptr_t argc_tag_offset = NativeArguments::argc_tag_offset();
const intptr_t argv_offset = NativeArguments::argv_offset();
const intptr_t retval_offset = NativeArguments::retval_offset();
+ const intptr_t exitframe_last_param_slot_from_fp = 1;
__ SetPrologueOffset();
__ TraceSimMsg("CallToRuntimeStub");
- __ EnterStubFrame();
+ __ EnterFrame();
// Load current Isolate pointer from Context structure into A0.
__ lw(A0, FieldAddress(CTX, Context::isolate_offset()));
@@ -77,7 +78,7 @@
__ sll(A2, S4, 2);
__ addu(A2, FP, A2); // Compute argv.
// Set argv in NativeArguments.
- __ addiu(A2, A2, Immediate(kParamEndSlotFromFp * kWordSize));
+ __ addiu(A2, A2, Immediate(exitframe_last_param_slot_from_fp * kWordSize));
ASSERT(retval_offset == 3 * kWordSize);
@@ -102,7 +103,7 @@
// Cache Context pointer into CTX while executing Dart code.
__ mov(CTX, A2);
- __ LeaveStubFrameAndReturn();
+ __ LeaveFrameAndReturn();
}
@@ -139,10 +140,7 @@
__ SetPrologueOffset();
__ TraceSimMsg("CallNativeCFunctionStub");
- __ addiu(SP, SP, Immediate(-2 * kWordSize));
- __ sw(RA, Address(SP, 1 * kWordSize));
- __ sw(FP, Address(SP, 0 * kWordSize));
- __ mov(FP, SP);
+ __ EnterFrame();
// Load current Isolate pointer from Context structure into A0.
__ lw(A0, FieldAddress(CTX, Context::isolate_offset()));
@@ -214,11 +212,7 @@
// Cache Context pointer into CTX while executing Dart code.
__ mov(CTX, A2);
- __ mov(SP, FP);
- __ lw(RA, Address(SP, 1 * kWordSize));
- __ lw(FP, Address(SP, 0 * kWordSize));
- __ Ret();
- __ delay_slot()->addiu(SP, SP, Immediate(2 * kWordSize));
+ __ LeaveFrameAndReturn();
}
@@ -236,10 +230,7 @@
__ SetPrologueOffset();
__ TraceSimMsg("CallNativeCFunctionStub");
- __ addiu(SP, SP, Immediate(-2 * kWordSize));
- __ sw(RA, Address(SP, 1 * kWordSize));
- __ sw(FP, Address(SP, 0 * kWordSize));
- __ mov(FP, SP);
+ __ EnterFrame();
// Load current Isolate pointer from Context structure into A0.
__ lw(A0, FieldAddress(CTX, Context::isolate_offset()));
@@ -302,11 +293,7 @@
// Cache Context pointer into CTX while executing Dart code.
__ mov(CTX, A2);
- __ mov(SP, FP);
- __ lw(RA, Address(SP, 1 * kWordSize));
- __ lw(FP, Address(SP, 0 * kWordSize));
- __ Ret();
- __ delay_slot()->addiu(SP, SP, Immediate(2 * kWordSize));
+ __ LeaveFrameAndReturn();
}
@@ -901,7 +888,7 @@
void StubCode::GenerateInvokeDartCodeStub(Assembler* assembler) {
// Save frame pointer coming in.
__ TraceSimMsg("InvokeDartCodeStub");
- __ EnterStubFrame();
+ __ EnterFrame();
// Save new context and C++ ABI callee-saved registers.
@@ -931,6 +918,11 @@
__ sw(A3, Address(SP, 2 * kWordSize));
+ // We now load the pool pointer(PP) as we are about to invoke dart code and we
+ // could potentially invoke some intrinsic functions which need the PP to be
+ // set up.
+ __ LoadPoolPointer();
+
// The new Context structure contains a pointer to the current Isolate
// structure. Cache the Context pointer in the CTX register so that it is
// available in generated code and calls to Isolate::Current() need not be
@@ -1034,7 +1026,7 @@
__ addiu(SP, SP, Immediate(kPreservedRegSpace));
// Restore the frame pointer and return.
- __ LeaveStubFrameAndReturn();
+ __ LeaveFrameAndReturn();
}
@@ -1408,7 +1400,7 @@
__ addiu(SP, SP, Immediate(4 * kWordSize)); // Pop arguments.
// V0: new object
// Restore the frame pointer and return.
- __ LeaveStubFrameAndReturn(RA, true);
+ __ LeaveStubFrameAndReturn(RA);
}
@@ -1516,7 +1508,7 @@
// V0: new object.
__ addiu(V0, T2, Immediate(kHeapObjectTag));
- __ LeaveStubFrameAndReturn(RA, true);
+ __ LeaveStubFrameAndReturn(RA);
__ Bind(&slow_case);
}
@@ -1552,7 +1544,7 @@
// V0: new object
// Restore the frame pointer.
- __ LeaveStubFrameAndReturn(RA, true);
+ __ LeaveStubFrameAndReturn(RA);
}
« no previous file with comments | « runtime/vm/stub_code_arm.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698