Revision: 15027
Author: [email protected]
Date: Mon Jun 10 02:12:57 2013
Log: Fix LoadIC calling convention on ARM
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 in
VisitYield.
[email protected]
BUG=
Review URL: https://codereview.chromium.org/16203004
http://code.google.com/p/v8/source/detail?r=15027
Modified:
/branches/bleeding_edge/src/arm/full-codegen-arm.cc
/branches/bleeding_edge/src/arm/ic-arm.cc
=======================================
--- /branches/bleeding_edge/src/arm/full-codegen-arm.cc Fri Jun 7 04:12:21
2013
+++ /branches/bleeding_edge/src/arm/full-codegen-arm.cc Mon Jun 10 02:12:57
2013
@@ -2041,11 +2041,9 @@
__ push(r3); // iter
__ push(r0); // exception
__ mov(r0, r3); // iter
- __ push(r0); // push LoadIC
state
__ LoadRoot(r2, Heap::kthrow_stringRootIndex); // "throw"
Handle<Code> throw_ic = isolate()->builtins()->LoadIC_Initialize();
CallIC(throw_ic); // iter.throw in
r0
- __ add(sp, sp, Operand(kPointerSize)); // drop LoadIC
state
__ jmp(&l_call);
// try { received = yield result.value }
@@ -2071,11 +2069,9 @@
__ push(r3); // iter
__ push(r0); // received
__ mov(r0, r3); // iter
- __ push(r0); // push LoadIC
state
__ LoadRoot(r2, Heap::ksend_stringRootIndex); // "send"
Handle<Code> send_ic = isolate()->builtins()->LoadIC_Initialize();
CallIC(send_ic); // iter.send in r0
- __ add(sp, sp, Operand(kPointerSize)); // drop LoadIC
state
// result = f.call(receiver, arg);
__ bind(&l_call);
@@ -2103,11 +2099,9 @@
__ pop(r1); // result
__ push(r0); // result.value
__ mov(r0, r1); // result
- __ push(r0); // push LoadIC
state
__ LoadRoot(r2, Heap::kdone_stringRootIndex); // "done"
Handle<Code> done_ic = isolate()->builtins()->LoadIC_Initialize();
CallIC(done_ic); // result.done in
r0
- __ add(sp, sp, Operand(kPointerSize)); // drop LoadIC
state
Handle<Code> bool_ic = ToBooleanStub::GetUninitialized(isolate());
CallIC(bool_ic);
__ cmp(r0, Operand(0));
=======================================
--- /branches/bleeding_edge/src/arm/ic-arm.cc Wed Jun 5 04:12:49 2013
+++ /branches/bleeding_edge/src/arm/ic-arm.cc Mon Jun 10 02:12:57 2013
@@ -651,7 +651,6 @@
// -- r2 : name
// -- lr : return address
// -- r0 : receiver
- // -- sp[0] : receiver
// -----------------------------------
// Probe the stub cache.
@@ -671,7 +670,6 @@
// -- r2 : name
// -- lr : return address
// -- r0 : receiver
- // -- sp[0] : receiver
// -----------------------------------
Label miss;
@@ -692,7 +690,6 @@
// -- r2 : name
// -- lr : return address
// -- r0 : receiver
- // -- sp[0] : receiver
// -----------------------------------
Isolate* isolate = masm->isolate();
@@ -713,7 +710,6 @@
// -- r2 : name
// -- lr : return address
// -- r0 : receiver
- // -- sp[0] : receiver
// -----------------------------------
__ mov(r3, r0);
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
