- Remove arguments definition test from the VM.

runtime/vm/flow_graph_builder.cc

Index: runtime/vm/flow_graph_builder.cc
===================================================================
--- runtime/vm/flow_graph_builder.cc	(revision 24474)
+++ runtime/vm/flow_graph_builder.cc	(working copy)
@@ -1813,17 +1813,6 @@
 }
 
 
-void EffectGraphVisitor::VisitArgumentDefinitionTestNode(
-    ArgumentDefinitionTestNode* node) {
-  InlineBailout("EffectGraphVisitor::VisitArgumentDefinitionTestNode");
-  Definition* load = BuildLoadLocal(node->saved_arguments_descriptor());
-  Value* arguments_descriptor = Bind(load);
-  ArgumentDefinitionTestInstr* arg_def_test =
-      new ArgumentDefinitionTestInstr(node, arguments_descriptor);
-  ReturnDefinition(arg_def_test);
-}
-
-
 intptr_t EffectGraphVisitor::GetCurrentTempLocalIndex() const {
   return kFirstLocalSlotFromFp
       - owner()->num_stack_locals()
@@ -3120,12 +3109,6 @@
       int num_params = function.NumParameters();

[regis, 2013/06/27 01:06:27]

Can be const again.

[Ivan Posva, 2013/06/27 18:36:44]

Done.

       int param_frame_index = (num_params == function.num_fixed_parameters()) ?
           (kParamEndSlotFromFp + num_params) : kFirstLocalSlotFromFp;
-      // Handle the saved arguments descriptor as an additional parameter.
-      if (owner()->parsed_function()->GetSavedArgumentsDescriptorVar() !=
-          NULL) {
-        ASSERT(param_frame_index == kFirstLocalSlotFromFp);
-        num_params++;
-      }
       for (int pos = 0; pos < num_params; param_frame_index--, pos++) {
         const LocalVariable& parameter = *scope->VariableAt(pos);
         ASSERT(parameter.owner() == scope);