Commit 3a1ef020 authored by mbrandy's avatar mbrandy Committed by Commit bot

PPC: [turbofan] Canonicalize return sequence for JSFunctions.

Port 2b9112a5

Original commit message:
This optimization is already implemented in fullcodegen, and
basically makes sure that we do not unecessarily blow up the
code with duplicated return sequences everywhere.

R=bmeurer@chromium.org, dstence@us.ibm.com, michael_dawson@ca.ibm.com
BUG=

Review URL: https://codereview.chromium.org/1218493003

Cr-Commit-Position: refs/heads/master@{#29333}
parent 3bdbb843
...@@ -1372,11 +1372,17 @@ void CodeGenerator::AssembleReturn() { ...@@ -1372,11 +1372,17 @@ void CodeGenerator::AssembleReturn() {
__ LeaveFrame(StackFrame::MANUAL); __ LeaveFrame(StackFrame::MANUAL);
__ Ret(); __ Ret();
} else if (descriptor->IsJSFunctionCall() || needs_frame_) { } else if (descriptor->IsJSFunctionCall() || needs_frame_) {
// Canonicalize JSFunction return sites for now.
if (return_label_.is_bound()) {
__ b(&return_label_);
} else {
__ bind(&return_label_);
int pop_count = descriptor->IsJSFunctionCall() int pop_count = descriptor->IsJSFunctionCall()
? static_cast<int>(descriptor->JSParameterCount()) ? static_cast<int>(descriptor->JSParameterCount())
: 0; : 0;
__ LeaveFrame(StackFrame::MANUAL, pop_count * kPointerSize); __ LeaveFrame(StackFrame::MANUAL, pop_count * kPointerSize);
__ Ret(); __ Ret();
}
} else { } else {
__ Ret(); __ Ret();
} }
......
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