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

[turbofan] Don't try to inline non-inlineable functions.

R=mstarzinger@chromium.org
BUG=chromium:540593
LOG=n

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

Cr-Commit-Position: refs/heads/master@{#31145}
parent 84065c5f
...@@ -254,6 +254,14 @@ Reduction JSInliner::Reduce(Node* node) { ...@@ -254,6 +254,14 @@ Reduction JSInliner::Reduce(Node* node) {
return NoChange(); return NoChange();
} }
if (!function->shared()->IsInlineable()) {
// Function must be inlineable.
TRACE("Not inlining %s into %s because callee is not inlineable\n",
function->shared()->DebugName()->ToCString().get(),
info_->shared_info()->DebugName()->ToCString().get());
return NoChange();
}
if (function->shared()->HasDebugInfo()) { if (function->shared()->HasDebugInfo()) {
// Function contains break points. // Function contains break points.
TRACE("Not inlining %s into %s because callee may contain break points\n", TRACE("Not inlining %s into %s because callee may contain break points\n",
......
// Copyright 2015 the V8 project authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
// Flags: --allow-natives-syntax --enable-slow-asserts --turbo-inlining
var __f_2 = (function(stdlib) {
"use asm";
var __v_3 = stdlib.Symbol;
function __f_2() { return __v_3(); }
return __f_2;
})(this);
%OptimizeFunctionOnNextCall(__f_2);
__f_2();
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