Generalize builtins inlining flag to allow forced inlining of any function

src/compiler/js-inlining.cc

 // Copyright 2014 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.
 
 #include "src/compiler/js-inlining.h"
 
 #include "src/ast.h"
 #include "src/ast-numbering.h"
 #include "src/compiler/all-nodes.h"
 #include "src/compiler/ast-graph-builder.h"
@@ skipping 277 matching lines @@
 
 Reduction JSInliner::Reduce(Node* node) {
   if (node->opcode() != IrOpcode::kJSCallFunction) return NoChange();
 
   JSCallFunctionAccessor call(node);
   HeapObjectMatcher<JSFunction> match(call.jsfunction());
   if (!match.HasValue()) return NoChange();
 
   Handle<JSFunction> function = match.Value().handle();
   if (!function->IsJSFunction()) return NoChange();
-  if (mode_ == kBuiltinsInlining && !function->shared()->inline_builtin()) {
+  if (mode_ == kBuiltinsInlining && !function->shared()->force_inline()) {

[Michael Starzinger, 2015/05/20 10:37:35]

nit: Maybe s/kBuiltinsInlining/kRestrictedInlining/ or something like that?

[danno, 2015/05/20 12:19:19]

Done.

     return NoChange();
   }
 
   Zone zone;
   ParseInfo parse_info(&zone, function);
   CompilationInfo info(&parse_info);
 
   if (!Compiler::ParseAndAnalyze(info.parse_info())) return NoChange();
   if (!Compiler::EnsureDeoptimizationSupport(&info)) return NoChange();
 
@@ skipping 53 matching lines @@
       }
     }
   }
 
   return InlineCall(node, inlinee);
 }
 
 }  // namespace compiler
 }  // namespace internal
 }  // namespace v8