More general CHA-based inlining and devirtualization for precompiled code.

runtime/vm/block_scheduler.cc

Index: runtime/vm/block_scheduler.cc
diff --git a/runtime/vm/block_scheduler.cc b/runtime/vm/block_scheduler.cc
index a84a5d96300b070fb4959b719a97e53a7455fe1e..38dfabb1cb0e4c38c9b995012b99a41ae0df4eb3 100644
--- a/runtime/vm/block_scheduler.cc
+++ b/runtime/vm/block_scheduler.cc
@@ -60,6 +60,10 @@ void BlockScheduler::AssignEdgeWeights() const {
 
   const Array& ic_data_array = Array::Handle(flow_graph()->zone(),
       flow_graph()->parsed_function().function().ic_data_array());
+  if (ic_data_array.IsNull()) {
+    // Function was not compiled yet. No edge counters available.
+    return;

[srdjan, 2015/10/21 17:37:46]

When can this happen? In precompilation FLAG_emit_edge_counters is false.

[Florian Schneider, 2015/10/22 13:59:25]

Yes, not needed with precompilation. Left over check when I tried inlining
methods without code with lazy compilation.

+  }
   Array& edge_counters = Array::Handle();
   edge_counters ^= ic_data_array.At(0);