Use explicit push-argument for InstanceSetter instruction.

vm/flow_graph_builder.h

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #ifndef VM_FLOW_GRAPH_BUILDER_H_
 #define VM_FLOW_GRAPH_BUILDER_H_
 
 #include "vm/allocation.h"
 #include "vm/ast.h"
 #include "vm/growable_array.h"
@@ skipping 175 matching lines @@
   // Perform a type check on the given value and return it.
   Value* BuildAssignableValue(intptr_t token_pos,
                               Value* value,
                               const AbstractType& dst_type,
                               const String& dst_name);
 
   void BuildStoreIndexedValues(StoreIndexedNode* node,
                                Value** array,
                                Value** index,
                                Value** value);
-  void BuildInstanceSetterValues(InstanceSetterNode* node,
-                                 Value** receiver,
-                                 Value** value);
+  void BuildInstanceSetterArguments(
+      InstanceSetterNode* node,
+      ZoneGrowableArray<PushArgumentInstr*>* arguments,
+      bool is_used);

[srdjan, 2012/08/01 16:53:31]

Add comment describing what 'is_used' means, or even better rename it so it is
self-explanatory, e.g., result_is_needed.

[Florian Schneider, 2012/08/02 10:06:59]

Done.

 
   virtual void BuildTypeTest(ComparisonNode* node);
   virtual void BuildTypeCast(ComparisonNode* node);
 
   bool MustSaveRestoreContext(SequenceNode* node) const;
 
   // Moves parent context into the context register.
   void UnchainContext();
 
   void CloseFragment() { exit_ = NULL; }
@@ skipping 142 matching lines @@
   // Output parameters.
   TargetEntryInstr** true_successor_address_;
   TargetEntryInstr** false_successor_address_;
 
   intptr_t condition_token_pos_;
 };
 
 }  // namespace dart
 
 #endif  // VM_FLOW_GRAPH_BUILDER_H_