Emit stackmaps even in unoptimized code if slow path pushes untagged values.

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 6e9c0dfa6e12b8b951c8c38538bd6bef49023314..60bffbded1bed58553f744a638c416872789f05e 100644
--- a/runtime/vm/flow_graph_compiler.h
+++ b/runtime/vm/flow_graph_compiler.h
@@ -637,6 +637,13 @@ class FlowGraphCompiler : public ValueObject {
 
   intptr_t GetOptimizationThreshold() const;
 
+  StackmapTableBuilder* stackmap_table_builder() {
+    if (stackmap_table_builder_ == NULL) {
+      stackmap_table_builder_ = new StackmapTableBuilder();

[koda, 2015/03/03 17:57:48]

Memory leak?

We should also assert in the destructor that the stack maps were actually
finalized (if non-NULL).

[Vyacheslav Egorov (Google), 2015/03/03 18:01:07]

it's a ZoneAllocated object as most of the things in the compiler, so it can't
leak anywhere (thus neither does it have its destructor invoked)

+    }
+    return stackmap_table_builder_;
+  }
+
   Isolate* isolate_;
   Assembler* assembler_;
   const ParsedFunction& parsed_function_;