Store pointer instead of reference to LocalVariable in ast and flow graph.

runtime/vm/object.h

Index: runtime/vm/object.h
===================================================================
--- runtime/vm/object.h	(revision 10207)
+++ runtime/vm/object.h	(working copy)
@@ -1357,7 +1357,6 @@
   bool HasInstantiatedSignature() const;
 
   RawClass* owner() const { return raw_ptr()->owner_; }
-  void set_owner(const Class& value) const;
 
   RawAbstractType* result_type() const { return raw_ptr()->result_type_; }

[Ivan Posva, 2012/08/03 20:10:26]

This is a completely unrelated change. Please remove.

[regis, 2012/08/06 22:31:49]

As the title of this cl states, this is a follow up to revision 10203 to address
late comments by Matthias.

   void set_result_type(const AbstractType& value) const;
@@ -1630,6 +1629,7 @@
   void set_is_const(bool is_const) const;
   void set_is_external(bool value) const;
   void set_parent_function(const Function& value) const;
+  void set_owner(const Class& value) const;
   void set_token_pos(intptr_t value) const;
   void set_implicit_closure_function(const Function& value) const;
   static RawFunction* New();
@@ -1674,9 +1674,6 @@
   void set_value(const Instance& value) const;
 
   RawClass* owner() const { return raw_ptr()->owner_; }
-  void set_owner(const Class& value) const {
-    StorePointer(&raw_ptr()->owner_, value.raw());
-  }
 
   RawAbstractType* type() const  { return raw_ptr()->type_; }
   void set_type(const AbstractType& value) const;
@@ -1722,6 +1719,9 @@
   void set_is_const(bool value) const {
     raw_ptr()->is_const_ = value;
   }
+  void set_owner(const Class& value) const {
+    StorePointer(&raw_ptr()->owner_, value.raw());
+  }
   void set_token_pos(intptr_t token_pos) const {
     raw_ptr()->token_pos_ = token_pos;
   }