After sudying IrregExp compilation, it seems too complicated to do lazy local var descriptors for t…

runtime/vm/flow_graph_compiler.cc

Index: runtime/vm/flow_graph_compiler.cc
===================================================================
--- runtime/vm/flow_graph_compiler.cc	(revision 45626)
+++ runtime/vm/flow_graph_compiler.cc	(working copy)
@@ -928,7 +928,10 @@
   }
   LocalVarDescriptors& var_descs = LocalVarDescriptors::Handle();
   if (parsed_function().node_sequence() == NULL) {
-    // TODO(srdjan): Implement lazy local var descriptors if Irregexp functions.
+    // Eager local var descriptors computation for Irregexp function as it is
+    // complicated to factor out.
+    // TODO(srdjan): Consider canonicalizing and reusing the local var
+    // descriptor for IrregexpFunction.
     ASSERT(flow_graph().IsIrregexpFunction());
     var_descs = LocalVarDescriptors::New(1);
     RawLocalVarDescriptors::VarInfo info;
@@ -938,12 +941,6 @@
     info.end_pos = 0;
     info.set_index(parsed_function().current_context_var()->index());
     var_descs.SetVar(0, Symbols::CurrentContextVar(), &info);
-  } else {
-    if (FLAG_eager_info_computation) {
-      var_descs =
-          parsed_function_.node_sequence()->scope()->GetVarDescriptors(
-              parsed_function_.function());
-    }
   }
   code.set_var_descriptors(var_descs);
 }