Commit a5ca412e authored by bmeurer's avatar bmeurer Committed by Commit bot

[turbofan] Properly respect --max_inlining_levels with Ignition.

The logic to test whether we already reached --max_inlining_levels
when inlining into some optimized function only checked specifically
for FrameStateType::kJavaScriptFunction, and thereby didn't properly
account for FrameStateType::kInterpretedFunction, which is what we
see when we come in via the bytecode pipeline.

Review-Url: https://codereview.chromium.org/2329923002
Cr-Commit-Position: refs/heads/master@{#39328}
parent e151c660
...@@ -109,7 +109,7 @@ Reduction JSInliningHeuristic::Reduce(Node* node) { ...@@ -109,7 +109,7 @@ Reduction JSInliningHeuristic::Reduce(Node* node) {
frame_state->opcode() == IrOpcode::kFrameState; frame_state->opcode() == IrOpcode::kFrameState;
frame_state = NodeProperties::GetFrameStateInput(frame_state)) { frame_state = NodeProperties::GetFrameStateInput(frame_state)) {
FrameStateInfo const& frame_info = OpParameter<FrameStateInfo>(frame_state); FrameStateInfo const& frame_info = OpParameter<FrameStateInfo>(frame_state);
if (frame_info.type() == FrameStateType::kJavaScriptFunction) { if (FrameStateFunctionInfo::IsJSFunctionType(frame_info.type())) {
if (++level > FLAG_max_inlining_levels) { if (++level > FLAG_max_inlining_levels) {
TRACE( TRACE(
"Not considering call site #%d:%s, because inlining depth " "Not considering call site #%d:%s, because inlining depth "
......
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