[turbofan] Disable inlining of Cpp builtins in need of argument adaption

src/compiler/js-typed-lowering.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-typed-lowering.h"
 
 #include "src/builtins/builtins-utils.h"
 #include "src/code-factory.h"
 #include "src/compilation-dependencies.h"
 #include "src/compiler/access-builder.h"
@@ skipping 1580 matching lines @@
   if (target_type->IsConstant() &&
       target_type->AsConstant()->Value()->IsJSFunction()) {
     Handle<JSFunction> function =
         Handle<JSFunction>::cast(target_type->AsConstant()->Value());
     Handle<SharedFunctionInfo> shared(function->shared(), isolate());
     const int builtin_index = shared->construct_stub()->builtin_index();
     const bool is_builtin = (builtin_index != -1);
 
     CallDescriptor::Flags flags = CallDescriptor::kNeedsFrameState;
 
-    if (is_builtin && Builtins::HasCppImplementation(builtin_index)) {
+    if (is_builtin && Builtins::HasCppImplementation(builtin_index) &&
+        (shared->internal_formal_parameter_count() == arity ||
+         shared->internal_formal_parameter_count() ==
+             SharedFunctionInfo::kDontAdaptArgumentsSentinel)) {
       // Patch {node} to a direct CEntryStub call.
 
       // Load the context from the {target}.
       Node* context = effect = graph()->NewNode(
           simplified()->LoadField(AccessBuilder::ForJSFunctionContext()),
           target, effect, control);
       NodeProperties::ReplaceContextInput(node, context);
 
       // Update the effect dependency for the {node}.
       NodeProperties::ReplaceEffectInput(node, effect);
@@ skipping 91 matching lines @@
     NodeProperties::ReplaceEffectInput(node, effect);
 
     // Compute flags for the call.
     CallDescriptor::Flags flags = CallDescriptor::kNeedsFrameState;
     if (p.tail_call_mode() == TailCallMode::kAllow) {
       flags |= CallDescriptor::kSupportsTailCalls;
     }
 
     Node* new_target = jsgraph()->UndefinedConstant();
     Node* argument_count = jsgraph()->Int32Constant(arity);
-    if (is_builtin && Builtins::HasCppImplementation(builtin_index)) {
+    if (is_builtin && Builtins::HasCppImplementation(builtin_index) &&
+        (shared->internal_formal_parameter_count() == arity ||
+         shared->internal_formal_parameter_count() ==
+             SharedFunctionInfo::kDontAdaptArgumentsSentinel)) {
       // Patch {node} to a direct CEntryStub call.
       ReduceBuiltin(isolate(), jsgraph(), node, builtin_index, arity, flags);
     } else if (shared->internal_formal_parameter_count() == arity ||
                shared->internal_formal_parameter_count() ==
                    SharedFunctionInfo::kDontAdaptArgumentsSentinel) {
       // Patch {node} to a direct call.
       node->InsertInput(graph()->zone(), arity + 2, new_target);
       node->InsertInput(graph()->zone(), arity + 3, argument_count);
       NodeProperties::ChangeOp(node,
                                common()->Call(Linkage::GetJSCallDescriptor(
@@ skipping 495 matching lines @@
 }
 
 
 CompilationDependencies* JSTypedLowering::dependencies() const {
   return dependencies_;
 }
 
 }  // namespace compiler
 }  // namespace internal
 }  // namespace v8