Index: src/mips/builtins-mips.cc |
diff --git a/src/mips/builtins-mips.cc b/src/mips/builtins-mips.cc |
index 100195b58d48bb6a97871ce9238369cd4d7c7ad9..d3c392607d89fc3561fd7902523264fc722e6001 100644 |
--- a/src/mips/builtins-mips.cc |
+++ b/src/mips/builtins-mips.cc |
@@ -315,6 +315,36 @@ void Builtins::Generate_InOptimizationQueue(MacroAssembler* masm) { |
} |
+static void Generate_Runtime_NewObject(MacroAssembler* masm, |
+ bool create_memento, |
+ Register original_constructor, |
+ Label* count_incremented, |
+ Label* allocated) { |
+ if (create_memento) { |
+ // Get the cell or allocation site. |
+ __ lw(a2, MemOperand(sp, 2 * kPointerSize)); |
+ __ push(a2); |
+ } |
+ |
+ __ push(a1); // argument for Runtime_NewObject |
+ __ push(original_constructor); // original constructor |
+ if (create_memento) { |
+ __ CallRuntime(Runtime::kNewObjectWithAllocationSite, 3); |
+ } else { |
+ __ CallRuntime(Runtime::kNewObject, 2); |
+ } |
+ __ mov(t4, v0); |
+ |
+ // Runtime_NewObjectWithAllocationSite increments allocation count. |
+ // Skip the increment. |
+ if (create_memento) { |
+ __ jmp(count_incremented); |
+ } else { |
+ __ jmp(allocated); |
+ } |
+} |
+ |
+ |
static void Generate_JSConstructStubHelper(MacroAssembler* masm, |
bool is_api_function, |
bool create_memento) { |
@@ -322,6 +352,7 @@ static void Generate_JSConstructStubHelper(MacroAssembler* masm, |
// -- a0 : number of arguments |
// -- a1 : constructor function |
// -- a2 : allocation site or undefined |
+ // -- a3 : original constructor |
// -- ra : return address |
// -- sp[...]: constructor arguments |
// ----------------------------------- |
@@ -343,7 +374,7 @@ static void Generate_JSConstructStubHelper(MacroAssembler* masm, |
FrameScope scope(masm, StackFrame::CONSTRUCT); |
if (create_memento) { |
- __ AssertUndefinedOrAllocationSite(a2, a3); |
+ __ AssertUndefinedOrAllocationSite(a2, t0); |
__ push(a2); |
} |
@@ -351,7 +382,14 @@ static void Generate_JSConstructStubHelper(MacroAssembler* masm, |
__ sll(a0, a0, kSmiTagSize); // Tag arguments count. |
__ MultiPushReversed(a0.bit() | a1.bit()); |
- Label rt_call, allocated; |
+ Label rt_call, allocated, normal_new, count_incremented; |
+ __ Branch(&normal_new, eq, a1, Operand(a3)); |
+ |
+ // Original constructor and function are different. |
+ Generate_Runtime_NewObject(masm, create_memento, a3, &count_incremented, |
+ &allocated); |
+ __ bind(&normal_new); |
+ |
// Try to allocate the object without transitioning into C code. If any of |
// the preconditions is not met, the code bails out to the runtime call. |
if (FLAG_inline_new) { |
@@ -590,27 +628,8 @@ static void Generate_JSConstructStubHelper(MacroAssembler* masm, |
// Allocate the new receiver object using the runtime call. |
// a1: constructor function |
__ bind(&rt_call); |
- if (create_memento) { |
- // Get the cell or allocation site. |
- __ lw(a2, MemOperand(sp, 2 * kPointerSize)); |
- __ push(a2); |
- } |
- |
- __ push(a1); // Argument for Runtime_NewObject. |
- if (create_memento) { |
- __ CallRuntime(Runtime::kNewObjectWithAllocationSite, 2); |
- } else { |
- __ CallRuntime(Runtime::kNewObject, 1); |
- } |
- __ mov(t4, v0); |
- |
- // If we ended up using the runtime, and we want a memento, then the |
- // runtime call made it for us, and we shouldn't do create count |
- // increment. |
- Label count_incremented; |
- if (create_memento) { |
- __ jmp(&count_incremented); |
- } |
+ Generate_Runtime_NewObject(masm, create_memento, a1, &count_incremented, |
+ &allocated); |
// Receiver for constructor call allocated. |
// t4: JSObject |