[turbofan] escape analysis supports arguments object and rest elements

src/compiler/js-call-reducer.cc

Index: src/compiler/js-call-reducer.cc
diff --git a/src/compiler/js-call-reducer.cc b/src/compiler/js-call-reducer.cc
index a7c9c001bfc357e52bbc3667729a9763c5c0c8e1..162acbb3ec56f226e8ed904274234b1bd1ba1e8a 100644
--- a/src/compiler/js-call-reducer.cc
+++ b/src/compiler/js-call-reducer.cc
@@ -136,20 +136,6 @@ Reduction JSCallReducer::ReduceFunctionPrototypeApply(Node* node) {
     // the outermost function.
     Node* outer_state = frame_state->InputAt(kFrameStateOuterStateInput);
     if (outer_state->opcode() != IrOpcode::kFrameState) {
-      // TODO(jarin,bmeurer): Support the NewUnmappedArgumentsElement and
-      // NewRestParameterElements in the EscapeAnalysis and Deoptimizer
-      // instead, then we don't need this hack.
-      // Only works with zero formal parameters because of lacking deoptimizer
-      // support.
-      if (type != CreateArgumentsType::kRestParameter &&
-          formal_parameter_count == 0) {
-        // There are no other uses of the {arg_array} except in StateValues,
-        // so we just replace {arg_array} with a marker for the Deoptimizer
-        // that this refers to the arguments object.
-        Node* arguments = graph()->NewNode(common()->ArgumentsObjectState());
-        ReplaceWithValue(arg_array, arguments);
-      }
-
       // Reduce {node} to a JSCallForwardVarargs operation, which just
       // re-pushes the incoming arguments and calls the {target}.
       node->RemoveInput(0);  // Function.prototype.apply