Use zone allocated growable array to populate interim static and stub calls table, thus allocating …

runtime/vm/flow_graph_compiler.h

Index: runtime/vm/flow_graph_compiler.h
diff --git a/runtime/vm/flow_graph_compiler.h b/runtime/vm/flow_graph_compiler.h
index 536720e2ac7c4059318294e362ff030df763ae9a..89e35eebd21a90fe8e326af3357502733fd65b4d 100644
--- a/runtime/vm/flow_graph_compiler.h
+++ b/runtime/vm/flow_graph_compiler.h
@@ -662,6 +662,16 @@ class FlowGraphCompiler : public ValueObject {
   void FrameStateClear();
 #endif
 
+  struct StaticCallsStruct {
+    intptr_t offset;
+    const Function* function;

[koda, 2015/08/20 18:13:18]

For readability, please comment (and preferably also assert) that exactly one of
these is NULL.

[srdjan, 2015/08/20 21:37:45]

Discussed offline, adding comment.

+    const Code* code;
+    StaticCallsStruct(intptr_t offset_arg,
+                      const Function* function_arg,
+                      const Code* code_arg)
+        : offset(offset_arg), function(function_arg), code(code_arg) {}
+  };
+
   Isolate* isolate_;
   Zone* zone_;
   Assembler* assembler_;
@@ -683,11 +693,10 @@ class FlowGraphCompiler : public ValueObject {
   GrowableArray<BlockInfo*> block_info_;
   GrowableArray<CompilerDeoptInfo*> deopt_infos_;
   GrowableArray<SlowPathCode*> slow_path_code_;
-  // Stores: [code offset, function or null, null(code)].
   // Stores static call targets as well as stub targets.
   // TODO(srdjan): Evaluate if we should store allocation stub targets into a
   // separate table?
-  const GrowableObjectArray& static_calls_target_table_;
+  GrowableArray<StaticCallsStruct> static_calls_target_table_;
   const bool is_optimizing_;
   // Set to true if optimized code has IC calls.
   bool may_reoptimize_;