Avoid using ProcessArguments for static calls.

src/compiler/ast-graph-builder.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/ast-graph-builder.h"
 
 #include "src/compiler.h"
 #include "src/compiler/ast-loop-assignment-analyzer.h"
 #include "src/compiler/control-builders.h"
 #include "src/compiler/machine-operator.h"
@@ skipping 649 matching lines @@
         NewNode(javascript()->StrictEqual(), obj, jsgraph()->NullConstant());
     is_null.If(is_null_cond);
     is_null.Then();
     is_null.Else();
     // Convert object to jsobject.
     // PrepareForBailoutForId(stmt->PrepareId(), TOS_REG);
     obj = NewNode(javascript()->ToObject(), obj);
     PrepareFrameState(obj, stmt->ToObjectId(), OutputFrameStateCombine::Push());
     environment()->Push(obj);
     // TODO(dcarney): should do a fast enum cache check here to skip runtime.
-    environment()->Push(obj);
-    Node* cache_type = ProcessArguments(
-        javascript()->CallRuntime(Runtime::kGetPropertyNamesFast, 1), 1);
+    Node* cache_type = NewNode(
+        javascript()->CallRuntime(Runtime::kGetPropertyNamesFast, 1), obj);
     PrepareFrameState(cache_type, stmt->EnumId(),
                       OutputFrameStateCombine::Push());
     // TODO(dcarney): these next runtime calls should be removed in favour of
     //                a few simplified instructions.
-    environment()->Push(obj);
-    environment()->Push(cache_type);
-    Node* cache_pair =
-        ProcessArguments(javascript()->CallRuntime(Runtime::kForInInit, 2), 2);
+    Node* cache_pair = NewNode(
+        javascript()->CallRuntime(Runtime::kForInInit, 2), obj, cache_type);
     // cache_type may have been replaced.
     Node* cache_array = NewNode(common()->Projection(0), cache_pair);
     cache_type = NewNode(common()->Projection(1), cache_pair);
-    environment()->Push(cache_type);
-    environment()->Push(cache_array);
-    Node* cache_length = ProcessArguments(
-        javascript()->CallRuntime(Runtime::kForInCacheArrayLength, 2), 2);
+    Node* cache_length =
+        NewNode(javascript()->CallRuntime(Runtime::kForInCacheArrayLength, 2),
+                cache_type, cache_array);
     {
       // TODO(dcarney): this check is actually supposed to be for the
       //                empty enum case only.
       IfBuilder have_no_properties(this);
       Node* empty_array_cond = NewNode(javascript()->StrictEqual(),
                                        cache_length, jsgraph()->ZeroConstant());
       have_no_properties.If(empty_array_cond);
       have_no_properties.Then();
       // Pop obj and skip loop.
       environment()->Pop();
@@ skipping 44 matching lines @@
     IfBuilder test_should_filter(this);
     Node* should_filter_cond = NewNode(
         javascript()->StrictEqual(), should_filter, jsgraph()->TrueConstant());
     test_should_filter.If(should_filter_cond);
     test_should_filter.Then();
     value = environment()->Pop();
     Node* builtins = BuildLoadBuiltinsObject();
     Node* function = BuildLoadObjectField(
         builtins,
         JSBuiltinsObject::OffsetOfFunctionWithId(Builtins::FILTER_KEY));
-    // Callee.
-    environment()->Push(function);
-    // Receiver.
-    environment()->Push(obj);
-    // Args.
-    environment()->Push(value);
     // result is either the string key or Smi(0) indicating the property
     // is gone.
-    Node* res = ProcessArguments(
-        javascript()->CallFunction(3, NO_CALL_FUNCTION_FLAGS), 3);
+    Node* res = NewNode(javascript()->CallFunction(3, NO_CALL_FUNCTION_FLAGS),
+                        function, obj, value);
     // TODO(jarin): provide real bailout id.
     PrepareFrameState(res, BailoutId::None());
     Node* property_missing =
         NewNode(javascript()->StrictEqual(), res, jsgraph()->ZeroConstant());
     {
       IfBuilder is_property_missing(this);
       is_property_missing.If(property_missing);
       is_property_missing.Then();
       // Inc counter and continue.
       Node* index_inc =
@@ skipping 1667 matching lines @@
 
 BitVector* AstGraphBuilder::GetVariablesAssignedInLoop(
     IterationStatement* stmt) {
   if (loop_assignment_analysis_ == NULL) return NULL;
   return loop_assignment_analysis_->GetVariablesAssignedInLoop(stmt);
 }
 
 }  // namespace compiler
 }  // namespace internal
 }  // namespace v8