Introduce Goto instructions to the flow graph.

runtime/vm/il_printer.cc

Index: runtime/vm/il_printer.cc
diff --git a/runtime/vm/il_printer.cc b/runtime/vm/il_printer.cc
index b6b8fe6680fc3e7b6e9289fa12060378e4d0eb3d..8684c615e5bcc2b14ae78798743d2aea5675663a 100644
--- a/runtime/vm/il_printer.cc
+++ b/runtime/vm/il_printer.cc
@@ -378,7 +378,9 @@ void PhiInstr::PrintTo(BufferFormatter* f) const {
 
 
 void ParameterInstr::PrintTo(BufferFormatter* f) const {
-  f->Print("    v%d <- parameter(%d)", index());
+  f->Print("    v%d <- parameter(%d)",
+           (ssa_temp_index() != -1) ? ssa_temp_index() : temp_index(),

[srdjan, 2012/07/12 16:42:26]

Maybe introduce HasSSATemp(), since there are many places where we check.

[Kevin Millikin (Google), 2012/07/13 15:34:59]

Hmm.  There are a few places where we use it to tell whether we've done the SSA
translation or not, and a few places where we use it to tell if a value has uses
after the translation.

Testing for the index seems indirect in both cases, even with a predicate for
it.

I'll probably make a separate change to clean it up.  I prefer calling it a
'virtual register index' anyway.

+           index());
 }
 
 
@@ -424,6 +426,11 @@ void ReThrowInstr::PrintTo(BufferFormatter* f) const {
 }
 
 
+void GotoInstr::PrintTo(BufferFormatter* f) const {
+  f->Print("    goto %d", successor()->block_id());
+}
+
+
 void BranchInstr::PrintTo(BufferFormatter* f) const {
   f->Print("    %s ", DebugName());
   f->Print("if ");
@@ -675,6 +682,11 @@ void ReThrowInstr::PrintToVisualizer(BufferFormatter* f) const {
 }
 
 
+void GotoInstr::PrintToVisualizer(BufferFormatter* f) const {
+  f->Print("_ goto B%d", successor()->block_id());
+}
+
+
 void BranchInstr::PrintToVisualizer(BufferFormatter* f) const {
   f->Print("_ %s ", DebugName());
   f->Print("if ");