Fix an off-by-one error in deoptimization compression.

runtime/vm/deopt_instructions.h

 // Copyright (c) 2013, 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_DEOPT_INSTRUCTIONS_H_
 #define VM_DEOPT_INSTRUCTIONS_H_
 
 #include "vm/allocation.h"
 #include "vm/assembler.h"
 #include "vm/code_generator.h"
@@ skipping 184 matching lines @@
   // for materialization: class of the instance to allocate and field-value
   // pairs for initialization.
   // Emitted instructions are expected to follow fixed size section of frame
   // emitted first. This way they become a part of the bottom-most deoptimized
   // frame and are discoverable by GC.
   // At deoptimization they will be removed by the stub at the very end:
   // after they were used to materialize objects.
   // Returns the index of the next stack slot. Used for verification.
   intptr_t EmitMaterializationArguments(intptr_t to_index);
 
-  RawDeoptInfo* CreateDeoptInfo();
+  RawDeoptInfo* CreateDeoptInfo(const Array& deopt_table);

[Kevin Millikin (Google), 2013/09/13 13:46:18]

The table gets threaded around just for verification (decompression).  I don't
see a simple way around it.

 
   // Mark the actual start of the frame description after all materialization
   // instructions were emitted. Used for verification purposes.
   void MarkFrameStart() {
     ASSERT(frame_start_ == -1);
     frame_start_ = instructions_.length();
   }
 
  private:
   class TrieNode;
@@ skipping 50 matching lines @@
                        DeoptInfo* info,
                        Smi* reason);
 
  private:
   static const intptr_t kEntrySize = 3;
 };
 
 }  // namespace dart
 
 #endif  // VM_DEOPT_INSTRUCTIONS_H_