Add missing StorePointer/StoreSmi in ContextScope.

runtime/vm/raw_object.h

Index: runtime/vm/raw_object.h
===================================================================
--- runtime/vm/raw_object.h	(revision 41280)
+++ runtime/vm/raw_object.h	(working copy)
@@ -1215,6 +1215,12 @@
   }
   // Variable length data follows here.
   RawObject** data() { OPEN_ARRAY_START(RawObject*, RawObject*); }

[Ivan Posva, 2014/10/29 05:17:27]

Should we have the non-const data() accessor in this class at all. When is it
ever needed apart in the from()/to() calculation?

[koda, 2014/10/29 20:10:27]

Changed from/to to use VariableDescAddr.

PTAL

+  RawObject* const* data() const { OPEN_ARRAY_START(RawObject*, RawObject*); }
+  const VariableDesc* VariableDescAddr(intptr_t index) const {

[Ivan Posva, 2014/10/29 05:17:27]

Why did you have to move this functionality to RawContextScope from
ContextScope?

[koda, 2014/10/29 20:10:27]

I wanted to keep the tricky representation details of the
struct/pointers/padding issues contained in this file. For instance, the person
addressing the above TODO will likely want to change VariableDescAddr too.

(And in the updated CL it's also needed for from/to.)

+    ASSERT((index >= 0) && (index < num_variables_));
+    // data() points to the first component of the first descriptor.
+    return &(reinterpret_cast<const VariableDesc*>(data())[index]);
+  }
   RawObject** to(intptr_t num_vars) {
     const intptr_t data_length = num_vars * (sizeof(VariableDesc)/kWordSize);
     return reinterpret_cast<RawObject**>(&ptr()->data()[data_length - 1]);