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

Unified Diff: runtime/vm/assembler_mips.h

Issue 1343373003: Revert "VM: New calling convention for generated code." (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 5 years, 3 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/assembler_ia32.cc ('k') | runtime/vm/assembler_mips.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/assembler_mips.h
diff --git a/runtime/vm/assembler_mips.h b/runtime/vm/assembler_mips.h
index 9a1bc2c38d7f9a4e177dba7452aaaae1c04936b5..2ae9456faf16196d1ac873ef13399822eee9b21e 100644
--- a/runtime/vm/assembler_mips.h
+++ b/runtime/vm/assembler_mips.h
@@ -288,12 +288,21 @@ class Assembler : public ValueObject {
// Set up a stub frame so that the stack traversal code can easily identify
// a stub frame.
- void EnterStubFrame(intptr_t frame_size = 0);
+ void EnterStubFrame();
void LeaveStubFrame();
// A separate macro for when a Ret immediately follows, so that we can use
// the branch delay slot.
void LeaveStubFrameAndReturn(Register ra = RA);
+ // Instruction pattern from entrypoint is used in dart frame prologs
+ // to set up the frame and save a PC which can be used to figure out the
+ // RawInstruction object corresponding to the code running in the frame.
+ // See EnterDartFrame. There are 6 instructions before we know the PC.
+ static const intptr_t kEntryPointToPcMarkerOffset = 6 * Instr::kInstrSize;
+ static intptr_t EntryPointToPcMarkerOffset() {
+ return kEntryPointToPcMarkerOffset;
+ }
+
void UpdateAllocationStats(intptr_t cid,
Register temp_reg,
Heap::Space space,
@@ -903,10 +912,11 @@ class Assembler : public ValueObject {
SubuDetectOverflow(rd, rs, rd, ro);
}
- void Branch(const StubEntry& stub_entry, Register pp = PP);
+ void Branch(const StubEntry& stub_entry);
- void BranchLink(const ExternalLabel* label, Patchability patchable);
+ void BranchPatchable(const StubEntry& stub_entry);
+ void BranchLink(const ExternalLabel* label, Patchability patchable);
void BranchLink(const StubEntry& stub_entry,
Patchability patchable = kNotPatchable);
@@ -919,17 +929,15 @@ class Assembler : public ValueObject {
}
}
- void LoadPoolPointer(Register reg = PP) {
+ void LoadPoolPointer() {
ASSERT(!in_delay_slot_);
- CheckCodePointer();
- lw(reg, FieldAddress(CODE_REG, Code::object_pool_offset()));
- set_constant_pool_allowed(reg == PP);
+ GetNextPC(TMP); // TMP gets the address of the next instruction.
+ const intptr_t object_pool_pc_dist =
+ Instructions::HeaderSize() - Instructions::object_pool_offset() +
+ CodeSize();
+ lw(PP, Address(TMP, -object_pool_pc_dist));
}
- void CheckCodePointer();
-
- void RestoreCodePointer();
-
void LoadImmediate(Register rd, int32_t value) {
ASSERT(!in_delay_slot_);
if (Utils::IsInt(kImmBits, value)) {
@@ -1510,14 +1518,12 @@ class Assembler : public ValueObject {
void EnterCallRuntimeFrame(intptr_t frame_space);
void LeaveCallRuntimeFrame();
+ void LoadWordFromPoolOffset(Register rd, int32_t offset);
void LoadObject(Register rd, const Object& object);
void LoadUniqueObject(Register rd, const Object& object);
void LoadExternalLabel(Register rd,
const ExternalLabel* label,
Patchability patchable);
- void LoadFunctionFromCalleePool(Register dst,
- const Function& function,
- Register new_pp);
void LoadNativeEntry(Register rd,
const ExternalLabel* label,
Patchability patchable);
@@ -1569,8 +1575,8 @@ class Assembler : public ValueObject {
// enable easy access to the RawInstruction object of code corresponding
// to this frame.
void EnterDartFrame(intptr_t frame_size);
- void LeaveDartFrame(RestorePP restore_pp = kRestoreCallerPP);
- void LeaveDartFrameAndReturn(Register ra = RA);
+ void LeaveDartFrame();
+ void LeaveDartFrameAndReturn();
// Set up a Dart frame for a function compiled for on-stack replacement.
// The frame layout is a normal Dart frame, but the frame is partially set
@@ -1631,9 +1637,7 @@ class Assembler : public ValueObject {
bool constant_pool_allowed_;
void BranchLink(const ExternalLabel* label);
- void BranchLink(const Code& code, Patchability patchable);
- void LoadWordFromPoolOffset(Register rd, int32_t offset, Register pp = PP);
void LoadObjectHelper(Register rd, const Object& object, bool is_unique);
void Emit(int32_t value) {
« no previous file with comments | « runtime/vm/assembler_ia32.cc ('k') | runtime/vm/assembler_mips.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698