Commit 2ebb9507 authored by palfia@homejinni.com's avatar palfia@homejinni.com

MIPS: Fix LoadIC calling convention.

Port r15027 (3ffb343)

Original commit message:
The comments in ic-arm.cc::LoadIC indicated that the receiver should be
both in a register and on the stack. This isn't true in fact: the code
is careful to spill the receiver if needed.

This CL also fixes up a mistaken use of this convention in VisitYield.

BUG=

Review URL: https://codereview.chromium.org/16131004
Patch from Balazs Kilvady <kilvadyb@homejinni.com>.

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@15048 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
parent b7e06629
...@@ -2049,12 +2049,10 @@ void FullCodeGenerator::VisitYield(Yield* expr) { ...@@ -2049,12 +2049,10 @@ void FullCodeGenerator::VisitYield(Yield* expr) {
__ push(a3); // iter __ push(a3); // iter
__ push(a0); // exception __ push(a0); // exception
__ mov(a0, a3); // iter __ mov(a0, a3); // iter
__ push(a0); // push LoadIC state
__ LoadRoot(a2, Heap::kthrow_stringRootIndex); // "throw" __ LoadRoot(a2, Heap::kthrow_stringRootIndex); // "throw"
Handle<Code> throw_ic = isolate()->builtins()->LoadIC_Initialize(); Handle<Code> throw_ic = isolate()->builtins()->LoadIC_Initialize();
CallIC(throw_ic); // iter.throw in a0 CallIC(throw_ic); // iter.throw in a0
__ mov(a0, v0); __ mov(a0, v0);
__ Addu(sp, sp, Operand(kPointerSize)); // drop LoadIC state
__ jmp(&l_call); __ jmp(&l_call);
// try { received = yield result.value } // try { received = yield result.value }
...@@ -2082,12 +2080,10 @@ void FullCodeGenerator::VisitYield(Yield* expr) { ...@@ -2082,12 +2080,10 @@ void FullCodeGenerator::VisitYield(Yield* expr) {
__ push(a3); // iter __ push(a3); // iter
__ push(a0); // received __ push(a0); // received
__ mov(a0, a3); // iter __ mov(a0, a3); // iter
__ push(a0); // push LoadIC state
__ LoadRoot(a2, Heap::ksend_stringRootIndex); // "send" __ LoadRoot(a2, Heap::ksend_stringRootIndex); // "send"
Handle<Code> send_ic = isolate()->builtins()->LoadIC_Initialize(); Handle<Code> send_ic = isolate()->builtins()->LoadIC_Initialize();
CallIC(send_ic); // iter.send in a0 CallIC(send_ic); // iter.send in a0
__ mov(a0, v0); __ mov(a0, v0);
__ Addu(sp, sp, Operand(kPointerSize)); // drop LoadIC state
// result = f.call(receiver, arg); // result = f.call(receiver, arg);
__ bind(&l_call); __ bind(&l_call);
...@@ -2117,11 +2113,9 @@ void FullCodeGenerator::VisitYield(Yield* expr) { ...@@ -2117,11 +2113,9 @@ void FullCodeGenerator::VisitYield(Yield* expr) {
__ pop(a1); // result __ pop(a1); // result
__ push(a0); // result.value __ push(a0); // result.value
__ mov(a0, a1); // result __ mov(a0, a1); // result
__ push(a0); // push LoadIC state
__ LoadRoot(a2, Heap::kdone_stringRootIndex); // "done" __ LoadRoot(a2, Heap::kdone_stringRootIndex); // "done"
Handle<Code> done_ic = isolate()->builtins()->LoadIC_Initialize(); Handle<Code> done_ic = isolate()->builtins()->LoadIC_Initialize();
CallIC(done_ic); // result.done in v0 CallIC(done_ic); // result.done in v0
__ Addu(sp, sp, Operand(kPointerSize)); // drop LoadIC state
__ mov(a0, v0); __ mov(a0, v0);
Handle<Code> bool_ic = ToBooleanStub::GetUninitialized(isolate()); Handle<Code> bool_ic = ToBooleanStub::GetUninitialized(isolate());
CallIC(bool_ic); CallIC(bool_ic);
......
...@@ -651,7 +651,6 @@ void LoadIC::GenerateMegamorphic(MacroAssembler* masm) { ...@@ -651,7 +651,6 @@ void LoadIC::GenerateMegamorphic(MacroAssembler* masm) {
// -- a2 : name // -- a2 : name
// -- ra : return address // -- ra : return address
// -- a0 : receiver // -- a0 : receiver
// -- sp[0] : receiver
// ----------------------------------- // -----------------------------------
// Probe the stub cache. // Probe the stub cache.
...@@ -671,7 +670,6 @@ void LoadIC::GenerateNormal(MacroAssembler* masm) { ...@@ -671,7 +670,6 @@ void LoadIC::GenerateNormal(MacroAssembler* masm) {
// -- a2 : name // -- a2 : name
// -- lr : return address // -- lr : return address
// -- a0 : receiver // -- a0 : receiver
// -- sp[0] : receiver
// ----------------------------------- // -----------------------------------
Label miss; Label miss;
...@@ -692,7 +690,6 @@ void LoadIC::GenerateMiss(MacroAssembler* masm) { ...@@ -692,7 +690,6 @@ void LoadIC::GenerateMiss(MacroAssembler* masm) {
// -- a2 : name // -- a2 : name
// -- ra : return address // -- ra : return address
// -- a0 : receiver // -- a0 : receiver
// -- sp[0] : receiver
// ----------------------------------- // -----------------------------------
Isolate* isolate = masm->isolate(); Isolate* isolate = masm->isolate();
...@@ -712,7 +709,6 @@ void LoadIC::GenerateRuntimeGetProperty(MacroAssembler* masm) { ...@@ -712,7 +709,6 @@ void LoadIC::GenerateRuntimeGetProperty(MacroAssembler* masm) {
// -- a2 : name // -- a2 : name
// -- ra : return address // -- ra : return address
// -- a0 : receiver // -- a0 : receiver
// -- sp[0] : receiver
// ----------------------------------- // -----------------------------------
__ mov(a3, a0); __ mov(a3, a0);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment