Chromium Code Reviews| Index: src/mips/macro-assembler-mips.cc | 
| diff --git a/src/mips/macro-assembler-mips.cc b/src/mips/macro-assembler-mips.cc | 
| index a85b0d803441b655e287e14ff66abe16e4b4712d..c42b57027284d9abf4edefbde2923946b8f3b191 100644 | 
| --- a/src/mips/macro-assembler-mips.cc | 
| +++ b/src/mips/macro-assembler-mips.cc | 
| @@ -3847,12 +3847,14 @@ static int AddressOffset(ExternalReference ref0, ExternalReference ref1) { | 
| } | 
| -void MacroAssembler::CallApiFunctionAndReturn(ExternalReference function, | 
| - Address function_address, | 
| - ExternalReference thunk_ref, | 
| - Register thunk_last_arg, | 
| - int stack_space, | 
| - int return_value_offset_from_fp) { | 
| +void MacroAssembler::CallApiFunctionAndReturn( | 
| + ExternalReference function, | 
| + Address function_address, | 
| + ExternalReference thunk_ref, | 
| + Register thunk_last_arg, | 
| + int stack_space, | 
| + MemOperand return_value_operand, | 
| + MemOperand* context_restore_operand) { | 
| ExternalReference next_address = | 
| ExternalReference::handle_scope_next_address(isolate()); | 
| const int kNextOffset = 0; | 
| @@ -3915,12 +3917,13 @@ void MacroAssembler::CallApiFunctionAndReturn(ExternalReference function, | 
| } | 
| Label promote_scheduled_exception; | 
| + Label exception_handled; | 
| Label delete_allocated_handles; | 
| Label leave_exit_frame; | 
| Label return_value_loaded; | 
| // Load value from ReturnValue. | 
| - lw(v0, MemOperand(fp, return_value_offset_from_fp*kPointerSize)); | 
| + lw(v0, return_value_operand); | 
| bind(&return_value_loaded); | 
| // No more valid handles (the result handle was the last one). Restore | 
| @@ -3941,14 +3944,23 @@ void MacroAssembler::CallApiFunctionAndReturn(ExternalReference function, | 
| li(at, Operand(ExternalReference::scheduled_exception_address(isolate()))); | 
| lw(t1, MemOperand(at)); | 
| Branch(&promote_scheduled_exception, ne, t0, Operand(t1)); | 
| + bind(&exception_handled); | 
| + | 
| + bool restore_context = context_restore_operand != NULL; | 
| + if (restore_context) { | 
| + lw(cp, *context_restore_operand); | 
| + } | 
| li(s0, Operand(stack_space)); | 
| - LeaveExitFrame(false, s0, true); | 
| + LeaveExitFrame(false, s0, !restore_context, true); | 
| 
 
Paul Lind
2013/09/17 17:54:22
Use new enum EMIT_RETURN.
 
kilvadyb
2013/09/17 18:40:42
Done.
 
 | 
| bind(&promote_scheduled_exception); | 
| - TailCallExternalReference( | 
| - ExternalReference(Runtime::kPromoteScheduledException, isolate()), | 
| - 0, | 
| - 1); | 
| + { | 
| + FrameScope frame(this, StackFrame::INTERNAL); | 
| + CallExternalReference( | 
| + ExternalReference(Runtime::kPromoteScheduledException, isolate()), | 
| + 0); | 
| + } | 
| + jmp(&exception_handled); | 
| // HandleScope limit has changed. Delete allocated extensions. | 
| bind(&delete_allocated_handles); | 
| @@ -4684,6 +4696,7 @@ void MacroAssembler::EnterExitFrame(bool save_doubles, | 
| void MacroAssembler::LeaveExitFrame(bool save_doubles, | 
| Register argument_count, | 
| + bool restore_context, | 
| bool do_return) { | 
| // Optionally restore all double registers. | 
| if (save_doubles) { | 
| @@ -4700,9 +4713,12 @@ void MacroAssembler::LeaveExitFrame(bool save_doubles, | 
| sw(zero_reg, MemOperand(t8)); | 
| // Restore current context from top and clear it in debug mode. | 
| - li(t8, Operand(ExternalReference(Isolate::kContextAddress, isolate()))); | 
| - lw(cp, MemOperand(t8)); | 
| + if (restore_context) { | 
| + li(t8, Operand(ExternalReference(Isolate::kContextAddress, isolate()))); | 
| + lw(cp, MemOperand(t8)); | 
| + } | 
| #ifdef DEBUG | 
| + li(t8, Operand(ExternalReference(Isolate::kContextAddress, isolate()))); | 
| sw(a3, MemOperand(t8)); | 
| #endif |