MIPS: Never record safepoint with doubles.
Port r22496 (f9dde0f) and r22498 (7a16d69) BUG= R=dusan.milosavljevic@imgtec.com Review URL: https://codereview.chromium.org/404143002 git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22527 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
This commit is contained in:
parent
9b3f0cbcb5
commit
47beb0c57a
@ -1138,11 +1138,7 @@ void ICCompareStub::GenerateGeneric(MacroAssembler* masm) {
|
||||
void StoreRegistersStateStub::Generate(MacroAssembler* masm) {
|
||||
__ mov(t9, ra);
|
||||
__ pop(ra);
|
||||
if (save_doubles_ == kSaveFPRegs) {
|
||||
__ PushSafepointRegistersAndDoubles();
|
||||
} else {
|
||||
__ PushSafepointRegisters();
|
||||
}
|
||||
__ PushSafepointRegisters();
|
||||
__ Jump(t9);
|
||||
}
|
||||
|
||||
@ -1150,12 +1146,7 @@ void StoreRegistersStateStub::Generate(MacroAssembler* masm) {
|
||||
void RestoreRegistersStateStub::Generate(MacroAssembler* masm) {
|
||||
__ mov(t9, ra);
|
||||
__ pop(ra);
|
||||
__ StoreToSafepointRegisterSlot(t9, t9);
|
||||
if (save_doubles_ == kSaveFPRegs) {
|
||||
__ PopSafepointRegistersAndDoubles();
|
||||
} else {
|
||||
__ PopSafepointRegisters();
|
||||
}
|
||||
__ PopSafepointRegisters();
|
||||
__ Jump(t9);
|
||||
}
|
||||
|
||||
@ -1421,23 +1412,15 @@ void CodeStub::GenerateStubsAheadOfTime(Isolate* isolate) {
|
||||
}
|
||||
|
||||
|
||||
void StoreRegistersStateStub::GenerateAheadOfTime(
|
||||
Isolate* isolate) {
|
||||
StoreRegistersStateStub stub1(isolate, kDontSaveFPRegs);
|
||||
stub1.GetCode();
|
||||
// Hydrogen code stubs need stub2 at snapshot time.
|
||||
StoreRegistersStateStub stub2(isolate, kSaveFPRegs);
|
||||
stub2.GetCode();
|
||||
void StoreRegistersStateStub::GenerateAheadOfTime(Isolate* isolate) {
|
||||
StoreRegistersStateStub stub(isolate);
|
||||
stub.GetCode();
|
||||
}
|
||||
|
||||
|
||||
void RestoreRegistersStateStub::GenerateAheadOfTime(
|
||||
Isolate* isolate) {
|
||||
RestoreRegistersStateStub stub1(isolate, kDontSaveFPRegs);
|
||||
stub1.GetCode();
|
||||
// Hydrogen code stubs need stub2 at snapshot time.
|
||||
RestoreRegistersStateStub stub2(isolate, kSaveFPRegs);
|
||||
stub2.GetCode();
|
||||
void RestoreRegistersStateStub::GenerateAheadOfTime(Isolate* isolate) {
|
||||
RestoreRegistersStateStub stub(isolate);
|
||||
stub.GetCode();
|
||||
}
|
||||
|
||||
|
||||
|
@ -78,28 +78,26 @@ class SubStringStub: public PlatformCodeStub {
|
||||
|
||||
class StoreRegistersStateStub: public PlatformCodeStub {
|
||||
public:
|
||||
explicit StoreRegistersStateStub(Isolate* isolate, SaveFPRegsMode with_fp)
|
||||
: PlatformCodeStub(isolate), save_doubles_(with_fp) {}
|
||||
explicit StoreRegistersStateStub(Isolate* isolate)
|
||||
: PlatformCodeStub(isolate) {}
|
||||
|
||||
static void GenerateAheadOfTime(Isolate* isolate);
|
||||
private:
|
||||
Major MajorKey() const { return StoreRegistersState; }
|
||||
int MinorKey() const { return (save_doubles_ == kSaveFPRegs) ? 1 : 0; }
|
||||
SaveFPRegsMode save_doubles_;
|
||||
int MinorKey() const { return 0; }
|
||||
|
||||
void Generate(MacroAssembler* masm);
|
||||
};
|
||||
|
||||
class RestoreRegistersStateStub: public PlatformCodeStub {
|
||||
public:
|
||||
explicit RestoreRegistersStateStub(Isolate* isolate, SaveFPRegsMode with_fp)
|
||||
: PlatformCodeStub(isolate), save_doubles_(with_fp) {}
|
||||
explicit RestoreRegistersStateStub(Isolate* isolate)
|
||||
: PlatformCodeStub(isolate) {}
|
||||
|
||||
static void GenerateAheadOfTime(Isolate* isolate);
|
||||
private:
|
||||
Major MajorKey() const { return RestoreRegistersState; }
|
||||
int MinorKey() const { return (save_doubles_ == kSaveFPRegs) ? 1 : 0; }
|
||||
SaveFPRegsMode save_doubles_;
|
||||
int MinorKey() const { return 0; }
|
||||
|
||||
void Generate(MacroAssembler* masm);
|
||||
};
|
||||
|
@ -1016,15 +1016,6 @@ void LCodeGen::RecordSafepointWithRegisters(LPointerMap* pointers,
|
||||
}
|
||||
|
||||
|
||||
void LCodeGen::RecordSafepointWithRegistersAndDoubles(
|
||||
LPointerMap* pointers,
|
||||
int arguments,
|
||||
Safepoint::DeoptMode deopt_mode) {
|
||||
RecordSafepoint(
|
||||
pointers, Safepoint::kWithRegistersAndDoubles, arguments, deopt_mode);
|
||||
}
|
||||
|
||||
|
||||
void LCodeGen::RecordAndWritePosition(int position) {
|
||||
if (position == RelocInfo::kNoPosition) return;
|
||||
masm()->positions_recorder()->RecordPosition(position);
|
||||
@ -2803,7 +2794,7 @@ void LCodeGen::DoDeferredInstanceOfKnownGlobal(LInstanceOfKnownGlobal* instr,
|
||||
flags | InstanceofStub::kReturnTrueFalseObject);
|
||||
InstanceofStub stub(isolate(), flags);
|
||||
|
||||
PushSafepointRegistersScope scope(this, Safepoint::kWithRegisters);
|
||||
PushSafepointRegistersScope scope(this);
|
||||
LoadContextFromDeferred(instr->context());
|
||||
|
||||
// Get the temp register reserved by the instruction. This needs to be t0 as
|
||||
@ -3654,7 +3645,7 @@ void LCodeGen::DoDeferredMathAbsTaggedHeapNumber(LMathAbs* instr) {
|
||||
// Input is negative. Reverse its sign.
|
||||
// Preserve the value of all registers.
|
||||
{
|
||||
PushSafepointRegistersScope scope(this, Safepoint::kWithRegisters);
|
||||
PushSafepointRegistersScope scope(this);
|
||||
|
||||
// Registers were saved at the safepoint, so we can use
|
||||
// many scratch registers.
|
||||
@ -4456,13 +4447,12 @@ void LCodeGen::DoTransitionElementsKind(LTransitionElementsKind* instr) {
|
||||
} else {
|
||||
ASSERT(object_reg.is(a0));
|
||||
ASSERT(ToRegister(instr->context()).is(cp));
|
||||
PushSafepointRegistersScope scope(
|
||||
this, Safepoint::kWithRegistersAndDoubles);
|
||||
PushSafepointRegistersScope scope(this);
|
||||
__ li(a1, Operand(to_map));
|
||||
bool is_js_array = from_map->instance_type() == JS_ARRAY_TYPE;
|
||||
TransitionElementsKindStub stub(isolate(), from_kind, to_kind, is_js_array);
|
||||
__ CallStub(&stub);
|
||||
RecordSafepointWithRegistersAndDoubles(
|
||||
RecordSafepointWithRegisters(
|
||||
instr->pointer_map(), 0, Safepoint::kLazyDeopt);
|
||||
}
|
||||
__ bind(¬_applicable);
|
||||
@ -4525,7 +4515,7 @@ void LCodeGen::DoDeferredStringCharCodeAt(LStringCharCodeAt* instr) {
|
||||
// contained in the register pointer map.
|
||||
__ mov(result, zero_reg);
|
||||
|
||||
PushSafepointRegistersScope scope(this, Safepoint::kWithRegisters);
|
||||
PushSafepointRegistersScope scope(this);
|
||||
__ push(string);
|
||||
// Push the index as a smi. This is safe because of the checks in
|
||||
// DoStringCharCodeAt above.
|
||||
@ -4589,7 +4579,7 @@ void LCodeGen::DoDeferredStringCharFromCode(LStringCharFromCode* instr) {
|
||||
// contained in the register pointer map.
|
||||
__ mov(result, zero_reg);
|
||||
|
||||
PushSafepointRegistersScope scope(this, Safepoint::kWithRegisters);
|
||||
PushSafepointRegistersScope scope(this);
|
||||
__ SmiTag(char_code);
|
||||
__ push(char_code);
|
||||
CallRuntimeFromDeferred(Runtime::kCharFromCode, 1, instr, instr->context());
|
||||
@ -4722,7 +4712,7 @@ void LCodeGen::DoDeferredNumberTagIU(LInstruction* instr,
|
||||
__ mov(dst, zero_reg);
|
||||
|
||||
// Preserve the value of all registers.
|
||||
PushSafepointRegistersScope scope(this, Safepoint::kWithRegisters);
|
||||
PushSafepointRegistersScope scope(this);
|
||||
|
||||
// NumberTagI and NumberTagD use the context from the frame, rather than
|
||||
// the environment's HContext or HInlinedContext value.
|
||||
@ -4788,7 +4778,7 @@ void LCodeGen::DoDeferredNumberTagD(LNumberTagD* instr) {
|
||||
Register reg = ToRegister(instr->result());
|
||||
__ mov(reg, zero_reg);
|
||||
|
||||
PushSafepointRegistersScope scope(this, Safepoint::kWithRegisters);
|
||||
PushSafepointRegistersScope scope(this);
|
||||
// NumberTagI and NumberTagD use the context from the frame, rather than
|
||||
// the environment's HContext or HInlinedContext value.
|
||||
// They only call Runtime::kAllocateHeapNumber.
|
||||
@ -5167,7 +5157,7 @@ void LCodeGen::DoCheckValue(LCheckValue* instr) {
|
||||
|
||||
void LCodeGen::DoDeferredInstanceMigration(LCheckMaps* instr, Register object) {
|
||||
{
|
||||
PushSafepointRegistersScope scope(this, Safepoint::kWithRegisters);
|
||||
PushSafepointRegistersScope scope(this);
|
||||
__ push(object);
|
||||
__ mov(cp, zero_reg);
|
||||
__ CallRuntimeSaveDoubles(Runtime::kTryMigrateInstance);
|
||||
@ -5379,7 +5369,7 @@ void LCodeGen::DoDeferredAllocate(LAllocate* instr) {
|
||||
// contained in the register pointer map.
|
||||
__ mov(result, zero_reg);
|
||||
|
||||
PushSafepointRegistersScope scope(this, Safepoint::kWithRegisters);
|
||||
PushSafepointRegistersScope scope(this);
|
||||
if (instr->size()->IsRegister()) {
|
||||
Register size = ToRegister(instr->size());
|
||||
ASSERT(!size.is(result));
|
||||
@ -5708,7 +5698,7 @@ void LCodeGen::DoDummyUse(LDummyUse* instr) {
|
||||
|
||||
|
||||
void LCodeGen::DoDeferredStackCheck(LStackCheck* instr) {
|
||||
PushSafepointRegistersScope scope(this, Safepoint::kWithRegisters);
|
||||
PushSafepointRegistersScope scope(this);
|
||||
LoadContextFromDeferred(instr->context());
|
||||
__ CallRuntimeSaveDoubles(Runtime::kStackGuard);
|
||||
RecordSafepointWithLazyDeopt(
|
||||
@ -5850,7 +5840,7 @@ void LCodeGen::DoDeferredLoadMutableDouble(LLoadFieldByIndex* instr,
|
||||
Register result,
|
||||
Register object,
|
||||
Register index) {
|
||||
PushSafepointRegistersScope scope(this, Safepoint::kWithRegisters);
|
||||
PushSafepointRegistersScope scope(this);
|
||||
__ Push(object, index);
|
||||
__ mov(cp, zero_reg);
|
||||
__ CallRuntimeSaveDoubles(Runtime::kLoadMutableDouble);
|
||||
|
@ -269,9 +269,6 @@ class LCodeGen: public LCodeGenBase {
|
||||
void RecordSafepointWithRegisters(LPointerMap* pointers,
|
||||
int arguments,
|
||||
Safepoint::DeoptMode mode);
|
||||
void RecordSafepointWithRegistersAndDoubles(LPointerMap* pointers,
|
||||
int arguments,
|
||||
Safepoint::DeoptMode mode);
|
||||
|
||||
void RecordAndWritePosition(int position) V8_OVERRIDE;
|
||||
|
||||
@ -386,56 +383,24 @@ class LCodeGen: public LCodeGenBase {
|
||||
|
||||
Safepoint::Kind expected_safepoint_kind_;
|
||||
|
||||
class PushSafepointRegistersScope V8_FINAL BASE_EMBEDDED {
|
||||
class PushSafepointRegistersScope V8_FINAL BASE_EMBEDDED {
|
||||
public:
|
||||
PushSafepointRegistersScope(LCodeGen* codegen,
|
||||
Safepoint::Kind kind)
|
||||
explicit PushSafepointRegistersScope(LCodeGen* codegen)
|
||||
: codegen_(codegen) {
|
||||
ASSERT(codegen_->info()->is_calling());
|
||||
ASSERT(codegen_->expected_safepoint_kind_ == Safepoint::kSimple);
|
||||
codegen_->expected_safepoint_kind_ = kind;
|
||||
codegen_->expected_safepoint_kind_ = Safepoint::kWithRegisters;
|
||||
|
||||
switch (codegen_->expected_safepoint_kind_) {
|
||||
case Safepoint::kWithRegisters: {
|
||||
StoreRegistersStateStub stub1(codegen_->masm_->isolate(),
|
||||
kDontSaveFPRegs);
|
||||
codegen_->masm_->push(ra);
|
||||
codegen_->masm_->CallStub(&stub1);
|
||||
break;
|
||||
}
|
||||
case Safepoint::kWithRegistersAndDoubles: {
|
||||
StoreRegistersStateStub stub2(codegen_->masm_->isolate(),
|
||||
kSaveFPRegs);
|
||||
codegen_->masm_->push(ra);
|
||||
codegen_->masm_->CallStub(&stub2);
|
||||
break;
|
||||
}
|
||||
default:
|
||||
UNREACHABLE();
|
||||
}
|
||||
StoreRegistersStateStub stub(codegen_->isolate());
|
||||
codegen_->masm_->push(ra);
|
||||
codegen_->masm_->CallStub(&stub);
|
||||
}
|
||||
|
||||
~PushSafepointRegistersScope() {
|
||||
Safepoint::Kind kind = codegen_->expected_safepoint_kind_;
|
||||
ASSERT((kind & Safepoint::kWithRegisters) != 0);
|
||||
switch (kind) {
|
||||
case Safepoint::kWithRegisters: {
|
||||
RestoreRegistersStateStub stub1(codegen_->masm_->isolate(),
|
||||
kDontSaveFPRegs);
|
||||
codegen_->masm_->push(ra);
|
||||
codegen_->masm_->CallStub(&stub1);
|
||||
break;
|
||||
}
|
||||
case Safepoint::kWithRegistersAndDoubles: {
|
||||
RestoreRegistersStateStub stub2(codegen_->masm_->isolate(),
|
||||
kSaveFPRegs);
|
||||
codegen_->masm_->push(ra);
|
||||
codegen_->masm_->CallStub(&stub2);
|
||||
break;
|
||||
}
|
||||
default:
|
||||
UNREACHABLE();
|
||||
}
|
||||
ASSERT(codegen_->expected_safepoint_kind_ == Safepoint::kWithRegisters);
|
||||
RestoreRegistersStateStub stub(codegen_->isolate());
|
||||
codegen_->masm_->push(ra);
|
||||
codegen_->masm_->CallStub(&stub);
|
||||
codegen_->expected_safepoint_kind_ = Safepoint::kSimple;
|
||||
}
|
||||
|
||||
|
@ -118,32 +118,6 @@ void MacroAssembler::PopSafepointRegisters() {
|
||||
}
|
||||
|
||||
|
||||
void MacroAssembler::PushSafepointRegistersAndDoubles() {
|
||||
PushSafepointRegisters();
|
||||
Subu(sp, sp, Operand(FPURegister::NumAllocatableRegisters() * kDoubleSize));
|
||||
for (int i = 0; i < FPURegister::NumAllocatableRegisters(); i+=2) {
|
||||
FPURegister reg = FPURegister::FromAllocationIndex(i);
|
||||
sdc1(reg, MemOperand(sp, i * kDoubleSize));
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
void MacroAssembler::PopSafepointRegistersAndDoubles() {
|
||||
for (int i = 0; i < FPURegister::NumAllocatableRegisters(); i+=2) {
|
||||
FPURegister reg = FPURegister::FromAllocationIndex(i);
|
||||
ldc1(reg, MemOperand(sp, i * kDoubleSize));
|
||||
}
|
||||
Addu(sp, sp, Operand(FPURegister::NumAllocatableRegisters() * kDoubleSize));
|
||||
PopSafepointRegisters();
|
||||
}
|
||||
|
||||
|
||||
void MacroAssembler::StoreToSafepointRegistersAndDoublesSlot(Register src,
|
||||
Register dst) {
|
||||
sw(src, SafepointRegistersAndDoublesSlot(dst));
|
||||
}
|
||||
|
||||
|
||||
void MacroAssembler::StoreToSafepointRegisterSlot(Register src, Register dst) {
|
||||
sw(src, SafepointRegisterSlot(dst));
|
||||
}
|
||||
|
@ -711,12 +711,9 @@ class MacroAssembler: public Assembler {
|
||||
// RegList constant kSafepointSavedRegisters.
|
||||
void PushSafepointRegisters();
|
||||
void PopSafepointRegisters();
|
||||
void PushSafepointRegistersAndDoubles();
|
||||
void PopSafepointRegistersAndDoubles();
|
||||
// Store value in register src in the safepoint stack slot for
|
||||
// register dst.
|
||||
void StoreToSafepointRegisterSlot(Register src, Register dst);
|
||||
void StoreToSafepointRegistersAndDoublesSlot(Register src, Register dst);
|
||||
// Load the value of the src register from its safepoint stack slot
|
||||
// into register dst.
|
||||
void LoadFromSafepointRegisterSlot(Register dst, Register src);
|
||||
|
@ -1133,11 +1133,7 @@ void ICCompareStub::GenerateGeneric(MacroAssembler* masm) {
|
||||
void StoreRegistersStateStub::Generate(MacroAssembler* masm) {
|
||||
__ mov(t9, ra);
|
||||
__ pop(ra);
|
||||
if (save_doubles_ == kSaveFPRegs) {
|
||||
__ PushSafepointRegistersAndDoubles();
|
||||
} else {
|
||||
__ PushSafepointRegisters();
|
||||
}
|
||||
__ PushSafepointRegisters();
|
||||
__ Jump(t9);
|
||||
}
|
||||
|
||||
@ -1145,11 +1141,7 @@ void StoreRegistersStateStub::Generate(MacroAssembler* masm) {
|
||||
void RestoreRegistersStateStub::Generate(MacroAssembler* masm) {
|
||||
__ mov(t9, ra);
|
||||
__ pop(ra);
|
||||
if (save_doubles_ == kSaveFPRegs) {
|
||||
__ PopSafepointRegistersAndDoubles();
|
||||
} else {
|
||||
__ PopSafepointRegisters();
|
||||
}
|
||||
__ PopSafepointRegisters();
|
||||
__ Jump(t9);
|
||||
}
|
||||
|
||||
@ -1415,23 +1407,15 @@ void CodeStub::GenerateStubsAheadOfTime(Isolate* isolate) {
|
||||
}
|
||||
|
||||
|
||||
void StoreRegistersStateStub::GenerateAheadOfTime(
|
||||
Isolate* isolate) {
|
||||
StoreRegistersStateStub stub1(isolate, kDontSaveFPRegs);
|
||||
stub1.GetCode();
|
||||
// Hydrogen code stubs need stub2 at snapshot time.
|
||||
StoreRegistersStateStub stub2(isolate, kSaveFPRegs);
|
||||
stub2.GetCode();
|
||||
void StoreRegistersStateStub::GenerateAheadOfTime(Isolate* isolate) {
|
||||
StoreRegistersStateStub stub(isolate);
|
||||
stub.GetCode();
|
||||
}
|
||||
|
||||
|
||||
void RestoreRegistersStateStub::GenerateAheadOfTime(
|
||||
Isolate* isolate) {
|
||||
RestoreRegistersStateStub stub1(isolate, kDontSaveFPRegs);
|
||||
stub1.GetCode();
|
||||
// Hydrogen code stubs need stub2 at snapshot time.
|
||||
RestoreRegistersStateStub stub2(isolate, kSaveFPRegs);
|
||||
stub2.GetCode();
|
||||
void RestoreRegistersStateStub::GenerateAheadOfTime(Isolate* isolate) {
|
||||
RestoreRegistersStateStub stub(isolate);
|
||||
stub.GetCode();
|
||||
}
|
||||
|
||||
|
||||
|
@ -77,28 +77,26 @@ class SubStringStub: public PlatformCodeStub {
|
||||
|
||||
class StoreRegistersStateStub: public PlatformCodeStub {
|
||||
public:
|
||||
explicit StoreRegistersStateStub(Isolate* isolate, SaveFPRegsMode with_fp)
|
||||
: PlatformCodeStub(isolate), save_doubles_(with_fp) {}
|
||||
explicit StoreRegistersStateStub(Isolate* isolate)
|
||||
: PlatformCodeStub(isolate) {}
|
||||
|
||||
static void GenerateAheadOfTime(Isolate* isolate);
|
||||
private:
|
||||
Major MajorKey() const { return StoreRegistersState; }
|
||||
int MinorKey() const { return (save_doubles_ == kSaveFPRegs) ? 1 : 0; }
|
||||
SaveFPRegsMode save_doubles_;
|
||||
int MinorKey() const { return 0; }
|
||||
|
||||
void Generate(MacroAssembler* masm);
|
||||
};
|
||||
|
||||
class RestoreRegistersStateStub: public PlatformCodeStub {
|
||||
public:
|
||||
explicit RestoreRegistersStateStub(Isolate* isolate, SaveFPRegsMode with_fp)
|
||||
: PlatformCodeStub(isolate), save_doubles_(with_fp) {}
|
||||
explicit RestoreRegistersStateStub(Isolate* isolate)
|
||||
: PlatformCodeStub(isolate) {}
|
||||
|
||||
static void GenerateAheadOfTime(Isolate* isolate);
|
||||
private:
|
||||
Major MajorKey() const { return RestoreRegistersState; }
|
||||
int MinorKey() const { return (save_doubles_ == kSaveFPRegs) ? 1 : 0; }
|
||||
SaveFPRegsMode save_doubles_;
|
||||
int MinorKey() const { return 0; }
|
||||
|
||||
void Generate(MacroAssembler* masm);
|
||||
};
|
||||
|
@ -971,15 +971,6 @@ void LCodeGen::RecordSafepointWithRegisters(LPointerMap* pointers,
|
||||
}
|
||||
|
||||
|
||||
void LCodeGen::RecordSafepointWithRegistersAndDoubles(
|
||||
LPointerMap* pointers,
|
||||
int arguments,
|
||||
Safepoint::DeoptMode deopt_mode) {
|
||||
RecordSafepoint(
|
||||
pointers, Safepoint::kWithRegistersAndDoubles, arguments, deopt_mode);
|
||||
}
|
||||
|
||||
|
||||
void LCodeGen::RecordAndWritePosition(int position) {
|
||||
if (position == RelocInfo::kNoPosition) return;
|
||||
masm()->positions_recorder()->RecordPosition(position);
|
||||
@ -2771,7 +2762,7 @@ void LCodeGen::DoDeferredInstanceOfKnownGlobal(LInstanceOfKnownGlobal* instr,
|
||||
flags | InstanceofStub::kReturnTrueFalseObject);
|
||||
InstanceofStub stub(isolate(), flags);
|
||||
|
||||
PushSafepointRegistersScope scope(this, Safepoint::kWithRegisters);
|
||||
PushSafepointRegistersScope scope(this);
|
||||
LoadContextFromDeferred(instr->context());
|
||||
|
||||
// Get the temp register reserved by the instruction. This needs to be a4 as
|
||||
@ -3681,7 +3672,7 @@ void LCodeGen::DoDeferredMathAbsTaggedHeapNumber(LMathAbs* instr) {
|
||||
// Input is negative. Reverse its sign.
|
||||
// Preserve the value of all registers.
|
||||
{
|
||||
PushSafepointRegistersScope scope(this, Safepoint::kWithRegisters);
|
||||
PushSafepointRegistersScope scope(this);
|
||||
|
||||
// Registers were saved at the safepoint, so we can use
|
||||
// many scratch registers.
|
||||
@ -4518,13 +4509,12 @@ void LCodeGen::DoTransitionElementsKind(LTransitionElementsKind* instr) {
|
||||
} else {
|
||||
ASSERT(object_reg.is(a0));
|
||||
ASSERT(ToRegister(instr->context()).is(cp));
|
||||
PushSafepointRegistersScope scope(
|
||||
this, Safepoint::kWithRegistersAndDoubles);
|
||||
PushSafepointRegistersScope scope(this);
|
||||
__ li(a1, Operand(to_map));
|
||||
bool is_js_array = from_map->instance_type() == JS_ARRAY_TYPE;
|
||||
TransitionElementsKindStub stub(isolate(), from_kind, to_kind, is_js_array);
|
||||
__ CallStub(&stub);
|
||||
RecordSafepointWithRegistersAndDoubles(
|
||||
RecordSafepointWithRegisters(
|
||||
instr->pointer_map(), 0, Safepoint::kLazyDeopt);
|
||||
}
|
||||
__ bind(¬_applicable);
|
||||
@ -4587,7 +4577,7 @@ void LCodeGen::DoDeferredStringCharCodeAt(LStringCharCodeAt* instr) {
|
||||
// contained in the register pointer map.
|
||||
__ mov(result, zero_reg);
|
||||
|
||||
PushSafepointRegistersScope scope(this, Safepoint::kWithRegisters);
|
||||
PushSafepointRegistersScope scope(this);
|
||||
__ push(string);
|
||||
// Push the index as a smi. This is safe because of the checks in
|
||||
// DoStringCharCodeAt above.
|
||||
@ -4651,7 +4641,7 @@ void LCodeGen::DoDeferredStringCharFromCode(LStringCharFromCode* instr) {
|
||||
// contained in the register pointer map.
|
||||
__ mov(result, zero_reg);
|
||||
|
||||
PushSafepointRegistersScope scope(this, Safepoint::kWithRegisters);
|
||||
PushSafepointRegistersScope scope(this);
|
||||
__ SmiTag(char_code);
|
||||
__ push(char_code);
|
||||
CallRuntimeFromDeferred(Runtime::kCharFromCode, 1, instr, instr->context());
|
||||
@ -4755,7 +4745,7 @@ void LCodeGen::DoDeferredNumberTagIU(LInstruction* instr,
|
||||
// contains an integer value.
|
||||
__ mov(dst, zero_reg);
|
||||
// Preserve the value of all registers.
|
||||
PushSafepointRegistersScope scope(this, Safepoint::kWithRegisters);
|
||||
PushSafepointRegistersScope scope(this);
|
||||
|
||||
// NumberTagI and NumberTagD use the context from the frame, rather than
|
||||
// the environment's HContext or HInlinedContext value.
|
||||
@ -4818,7 +4808,7 @@ void LCodeGen::DoDeferredNumberTagD(LNumberTagD* instr) {
|
||||
Register reg = ToRegister(instr->result());
|
||||
__ mov(reg, zero_reg);
|
||||
|
||||
PushSafepointRegistersScope scope(this, Safepoint::kWithRegisters);
|
||||
PushSafepointRegistersScope scope(this);
|
||||
// NumberTagI and NumberTagD use the context from the frame, rather than
|
||||
// the environment's HContext or HInlinedContext value.
|
||||
// They only call Runtime::kAllocateHeapNumber.
|
||||
@ -5196,7 +5186,7 @@ void LCodeGen::DoCheckValue(LCheckValue* instr) {
|
||||
|
||||
void LCodeGen::DoDeferredInstanceMigration(LCheckMaps* instr, Register object) {
|
||||
{
|
||||
PushSafepointRegistersScope scope(this, Safepoint::kWithRegisters);
|
||||
PushSafepointRegistersScope scope(this);
|
||||
__ push(object);
|
||||
__ mov(cp, zero_reg);
|
||||
__ CallRuntimeSaveDoubles(Runtime::kTryMigrateInstance);
|
||||
@ -5408,7 +5398,7 @@ void LCodeGen::DoDeferredAllocate(LAllocate* instr) {
|
||||
// contained in the register pointer map.
|
||||
__ mov(result, zero_reg);
|
||||
|
||||
PushSafepointRegistersScope scope(this, Safepoint::kWithRegisters);
|
||||
PushSafepointRegistersScope scope(this);
|
||||
if (instr->size()->IsRegister()) {
|
||||
Register size = ToRegister(instr->size());
|
||||
ASSERT(!size.is(result));
|
||||
@ -5739,7 +5729,7 @@ void LCodeGen::DoDummyUse(LDummyUse* instr) {
|
||||
|
||||
|
||||
void LCodeGen::DoDeferredStackCheck(LStackCheck* instr) {
|
||||
PushSafepointRegistersScope scope(this, Safepoint::kWithRegisters);
|
||||
PushSafepointRegistersScope scope(this);
|
||||
LoadContextFromDeferred(instr->context());
|
||||
__ CallRuntimeSaveDoubles(Runtime::kStackGuard);
|
||||
RecordSafepointWithLazyDeopt(
|
||||
@ -5881,7 +5871,7 @@ void LCodeGen::DoDeferredLoadMutableDouble(LLoadFieldByIndex* instr,
|
||||
Register result,
|
||||
Register object,
|
||||
Register index) {
|
||||
PushSafepointRegistersScope scope(this, Safepoint::kWithRegisters);
|
||||
PushSafepointRegistersScope scope(this);
|
||||
__ Push(object, index);
|
||||
__ mov(cp, zero_reg);
|
||||
__ CallRuntimeSaveDoubles(Runtime::kLoadMutableDouble);
|
||||
|
@ -270,9 +270,6 @@ class LCodeGen: public LCodeGenBase {
|
||||
void RecordSafepointWithRegisters(LPointerMap* pointers,
|
||||
int arguments,
|
||||
Safepoint::DeoptMode mode);
|
||||
void RecordSafepointWithRegistersAndDoubles(LPointerMap* pointers,
|
||||
int arguments,
|
||||
Safepoint::DeoptMode mode);
|
||||
|
||||
void RecordAndWritePosition(int position) V8_OVERRIDE;
|
||||
|
||||
@ -387,56 +384,24 @@ class LCodeGen: public LCodeGenBase {
|
||||
|
||||
Safepoint::Kind expected_safepoint_kind_;
|
||||
|
||||
class PushSafepointRegistersScope V8_FINAL BASE_EMBEDDED {
|
||||
class PushSafepointRegistersScope V8_FINAL BASE_EMBEDDED {
|
||||
public:
|
||||
PushSafepointRegistersScope(LCodeGen* codegen,
|
||||
Safepoint::Kind kind)
|
||||
explicit PushSafepointRegistersScope(LCodeGen* codegen)
|
||||
: codegen_(codegen) {
|
||||
ASSERT(codegen_->info()->is_calling());
|
||||
ASSERT(codegen_->expected_safepoint_kind_ == Safepoint::kSimple);
|
||||
codegen_->expected_safepoint_kind_ = kind;
|
||||
codegen_->expected_safepoint_kind_ = Safepoint::kWithRegisters;
|
||||
|
||||
switch (codegen_->expected_safepoint_kind_) {
|
||||
case Safepoint::kWithRegisters: {
|
||||
StoreRegistersStateStub stub1(codegen_->masm_->isolate(),
|
||||
kDontSaveFPRegs);
|
||||
codegen_->masm_->push(ra);
|
||||
codegen_->masm_->CallStub(&stub1);
|
||||
break;
|
||||
}
|
||||
case Safepoint::kWithRegistersAndDoubles: {
|
||||
StoreRegistersStateStub stub2(codegen_->masm_->isolate(),
|
||||
kSaveFPRegs);
|
||||
codegen_->masm_->push(ra);
|
||||
codegen_->masm_->CallStub(&stub2);
|
||||
break;
|
||||
}
|
||||
default:
|
||||
UNREACHABLE();
|
||||
}
|
||||
StoreRegistersStateStub stub(codegen_->isolate());
|
||||
codegen_->masm_->push(ra);
|
||||
codegen_->masm_->CallStub(&stub);
|
||||
}
|
||||
|
||||
~PushSafepointRegistersScope() {
|
||||
Safepoint::Kind kind = codegen_->expected_safepoint_kind_;
|
||||
ASSERT((kind & Safepoint::kWithRegisters) != 0);
|
||||
switch (kind) {
|
||||
case Safepoint::kWithRegisters: {
|
||||
RestoreRegistersStateStub stub1(codegen_->masm_->isolate(),
|
||||
kDontSaveFPRegs);
|
||||
codegen_->masm_->push(ra);
|
||||
codegen_->masm_->CallStub(&stub1);
|
||||
break;
|
||||
}
|
||||
case Safepoint::kWithRegistersAndDoubles: {
|
||||
RestoreRegistersStateStub stub2(codegen_->masm_->isolate(),
|
||||
kSaveFPRegs);
|
||||
codegen_->masm_->push(ra);
|
||||
codegen_->masm_->CallStub(&stub2);
|
||||
break;
|
||||
}
|
||||
default:
|
||||
UNREACHABLE();
|
||||
}
|
||||
ASSERT(codegen_->expected_safepoint_kind_ == Safepoint::kWithRegisters);
|
||||
RestoreRegistersStateStub stub(codegen_->isolate());
|
||||
codegen_->masm_->push(ra);
|
||||
codegen_->masm_->CallStub(&stub);
|
||||
codegen_->expected_safepoint_kind_ = Safepoint::kSimple;
|
||||
}
|
||||
|
||||
|
@ -122,32 +122,6 @@ void MacroAssembler::PopSafepointRegisters() {
|
||||
}
|
||||
|
||||
|
||||
void MacroAssembler::PushSafepointRegistersAndDoubles() {
|
||||
PushSafepointRegisters();
|
||||
Dsubu(sp, sp, Operand(FPURegister::NumAllocatableRegisters() * kDoubleSize));
|
||||
for (int i = 0; i < FPURegister::NumAllocatableRegisters(); i++) {
|
||||
FPURegister reg = FPURegister::FromAllocationIndex(i);
|
||||
sdc1(reg, MemOperand(sp, i * kDoubleSize));
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
void MacroAssembler::PopSafepointRegistersAndDoubles() {
|
||||
for (int i = 0; i < FPURegister::NumAllocatableRegisters(); i++) {
|
||||
FPURegister reg = FPURegister::FromAllocationIndex(i);
|
||||
ldc1(reg, MemOperand(sp, i * kDoubleSize));
|
||||
}
|
||||
Daddu(sp, sp, Operand(FPURegister::NumAllocatableRegisters() * kDoubleSize));
|
||||
PopSafepointRegisters();
|
||||
}
|
||||
|
||||
|
||||
void MacroAssembler::StoreToSafepointRegistersAndDoublesSlot(Register src,
|
||||
Register dst) {
|
||||
sd(src, SafepointRegistersAndDoublesSlot(dst));
|
||||
}
|
||||
|
||||
|
||||
void MacroAssembler::StoreToSafepointRegisterSlot(Register src, Register dst) {
|
||||
sd(src, SafepointRegisterSlot(dst));
|
||||
}
|
||||
|
@ -746,12 +746,9 @@ class MacroAssembler: public Assembler {
|
||||
// RegList constant kSafepointSavedRegisters.
|
||||
void PushSafepointRegisters();
|
||||
void PopSafepointRegisters();
|
||||
void PushSafepointRegistersAndDoubles();
|
||||
void PopSafepointRegistersAndDoubles();
|
||||
// Store value in register src in the safepoint stack slot for
|
||||
// register dst.
|
||||
void StoreToSafepointRegisterSlot(Register src, Register dst);
|
||||
void StoreToSafepointRegistersAndDoublesSlot(Register src, Register dst);
|
||||
// Load the value of the src register from its safepoint stack slot
|
||||
// into register dst.
|
||||
void LoadFromSafepointRegisterSlot(Register dst, Register src);
|
||||
|
Loading…
Reference in New Issue
Block a user