Added type propagation from store to load locals, but only within a sequence node (not across bas...

vm/scopes.h

 // Copyright (c) 2011, 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_SCOPES_H_
 #define VM_SCOPES_H_
 
 #include "vm/allocation.h"
 #include "vm/assembler.h"
 #include "vm/growable_array.h"
@@ skipping 48 matching lines @@
   void set_index(int index) {
     ASSERT(!HasIndex());
     ASSERT(index != kUnitializedIndex_);
     index_ = index;
   }
 
   void set_invisible(bool value) {
     is_invisible_ = value;
   }
 
+  bool Equals(const LocalVariable& other) const {
+    return HasIndex() && other.HasIndex() &&
+        (index() == other.index());

[regis, 2011/12/13 00:31:44]

This is not correct. You need to check is_captured() and
owner()->context_level() as well.

[srdjan, 2011/12/13 00:39:38]

Thanks!

+  }
+
  private:
   static const int kUnitializedIndex_ = INT_MIN;
 
   const intptr_t token_index_;
   const String& name_;
   LocalScope* owner_;  // Local scope declaring this variable.
 
   const AbstractType& type_;  // Declaration type of local variable.
 
   bool is_final_;  // If true, this variable is readonly.
@@ skipping 201 matching lines @@
   int num_context_variables_;  // Only set if this scope is a context owner.
   GrowableArray<LocalVariable*> variables_;
   GrowableArray<SourceLabel*> labels_;
 
   DISALLOW_COPY_AND_ASSIGN(LocalScope);
 };
 
 }  // namespace dart
 
 #endif  // VM_SCOPES_H_