Remember all deopt reasons in ic_data, not just the last one.

runtime/vm/code_descriptors.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_CODE_DESCRIPTORS_H_
 #define VM_CODE_DESCRIPTORS_H_
 
 #include "vm/ast.h"
 #include "vm/code_generator.h"
 #include "vm/globals.h"
 #include "vm/growable_array.h"
 #include "vm/object.h"
 
 namespace dart {
 
 class DescriptorList : public ZoneAllocated {
  public:
   struct PcDesc {
     intptr_t pc_offset;        // PC offset value of the descriptor.
     PcDescriptors::Kind kind;  // Descriptor kind (kDeopt, kOther).
     intptr_t deopt_id;         // Deoptimization id.
-    intptr_t data;             // Token position or deopt rason.
+    intptr_t data;             // Token position or deopt reason.
     intptr_t try_index;        // Try block index of PC or deopt array index.
     void SetTokenPos(intptr_t value) { data = value; }
     intptr_t TokenPos() const { return data; }
     void SetDeoptReason(DeoptReasonId value) { data = value; }
     DeoptReasonId DeoptReason() const {
+      ASSERT(static_cast<uword>(data) < kDeoptNumReasons);

[Ivan Posva, 2014/04/25 20:59:22]

Can we change this into a compile time assert?

[regis, 2014/04/25 23:38:28]

The value of 'data' is not known at compile time.

       return static_cast<DeoptReasonId>(data);
     }
   };
 
   explicit DescriptorList(intptr_t initial_capacity) : list_(initial_capacity) {
   }
   ~DescriptorList() { }
 
   intptr_t Length() const {
     return list_.length();
@@ skipping 139 matching lines @@
   }
 
  private:
   GrowableArray<struct HandlerDesc> list_;
   DISALLOW_COPY_AND_ASSIGN(ExceptionHandlerList);
 };
 
 }  // namespace dart
 
 #endif  // VM_CODE_DESCRIPTORS_H_