Index: src/builtins/ia32/builtins-ia32.cc |
diff --git a/src/builtins/ia32/builtins-ia32.cc b/src/builtins/ia32/builtins-ia32.cc |
index b549ab1336e4b3f2bb875fb446079e878e7f4a07..0d62f9f5b74e9483a4993c2d8af1a71e67a288ba 100644 |
--- a/src/builtins/ia32/builtins-ia32.cc |
+++ b/src/builtins/ia32/builtins-ia32.cc |
@@ -2968,9 +2968,16 @@ void Builtins::Generate_HandleFastApiCall(MacroAssembler* masm) { |
} |
} |
-void Builtins::Generate_OnStackReplacement(MacroAssembler* masm) { |
+static void Generate_OnStackReplacementHelper(MacroAssembler* masm, |
+ bool has_handler_frame) { |
// Lookup the function in the JavaScript frame. |
- __ mov(eax, Operand(ebp, JavaScriptFrameConstants::kFunctionOffset)); |
+ if (has_handler_frame) { |
+ __ mov(eax, Operand(ebp, StandardFrameConstants::kCallerFPOffset)); |
+ __ mov(eax, Operand(eax, JavaScriptFrameConstants::kFunctionOffset)); |
+ } else { |
+ __ mov(eax, Operand(ebp, JavaScriptFrameConstants::kFunctionOffset)); |
+ } |
+ |
{ |
FrameScope scope(masm, StackFrame::INTERNAL); |
// Pass function as argument. |
@@ -2979,13 +2986,19 @@ void Builtins::Generate_OnStackReplacement(MacroAssembler* masm) { |
} |
Label skip; |
- // If the code object is null, just return to the unoptimized code. |
+ // If the code object is null, just return to the caller. |
__ cmp(eax, Immediate(0)); |
__ j(not_equal, &skip, Label::kNear); |
__ ret(0); |
__ bind(&skip); |
+ // Drop any potential handler frame that is be sitting on top of the actual |
+ // JavaScript frame. This is the case then OSR is triggered from bytecode. |
+ if (has_handler_frame) { |
+ __ leave(); |
+ } |
+ |
// Load deoptimization data from the code object. |
__ mov(ebx, Operand(eax, Code::kDeoptimizationDataOffset - kHeapObjectTag)); |
@@ -3005,6 +3018,14 @@ void Builtins::Generate_OnStackReplacement(MacroAssembler* masm) { |
__ ret(0); |
} |
+void Builtins::Generate_OnStackReplacement(MacroAssembler* masm) { |
+ Generate_OnStackReplacementHelper(masm, false); |
+} |
+ |
+void Builtins::Generate_InterpreterOnStackReplacement(MacroAssembler* masm) { |
+ Generate_OnStackReplacementHelper(masm, true); |
+} |
+ |
#undef __ |
} // namespace internal |
} // namespace v8 |