Whenever possible use length passed to the List constructor for bounds checks instead of loading it…

runtime/vm/flow_graph_builder.cc

Index: runtime/vm/flow_graph_builder.cc
diff --git a/runtime/vm/flow_graph_builder.cc b/runtime/vm/flow_graph_builder.cc
index 927ee050ef0c1d2a26ff190836777ab951b7e3fa..0026c74547986f04a547c4b70a61bd9b27d06e35 100644
--- a/runtime/vm/flow_graph_builder.cc
+++ b/runtime/vm/flow_graph_builder.cc
@@ -1688,6 +1688,7 @@ void EffectGraphVisitor::VisitConstructorCallNode(ConstructorCallNode* node) {
                             arguments);
     // List factories return kArrayCid or kGrowableObjectArrayCid.
     call->set_result_cid(GetResultCidOfConstructor(node));
+    call->set_is_constructor(true);

[srdjan, 2012/09/12 10:29:58]

There is a Function::IsConstructor() and ::IsFactory() which you may use since
the StaticCallIInstr keeps the function.

     ReturnDefinition(call);
     return;
   }