Added flag --inline_smi_string_hashcode and --inline_smi_string_hashcode_ratio to decide when/if sh…

runtime/vm/il_printer.h

Index: runtime/vm/il_printer.h
diff --git a/runtime/vm/il_printer.h b/runtime/vm/il_printer.h
index bc19c75d1b9d76a42d51cc6f59c568d764cf584a..acfd16c6c635a42ecd7b862562d12ecfab368eee 100644
--- a/runtime/vm/il_printer.h
+++ b/runtime/vm/il_printer.h
@@ -57,8 +57,10 @@ class FlowGraphPrinter : public ValueObject {
 
   static void PrintGraph(const char* phase, FlowGraph* flow_graph);
 
-  // Debugging helper function.
-  static void PrintICData(const ICData& ic_data);
+  // Debugging helper function. If 'num_checks_to_print' is not specified
+  // or is negative, all checks will be printed.
+  static void PrintICData(const ICData& ic_data,
+                          intptr_t num_checks_to_print = -1);

[zra, 2016/04/27 16:28:17]

It looks like this function only previously had one call-site. Maybe instead of
using a default parameter we could have a:

static const intptr_t kPrintAll = -1;

[srdjan, 2016/04/27 18:10:22]

Using kPrintAll, but leaving it as default as I often use PrintICData as a
temporary debugging call and do not want to pass kPrintAll or -1 (from lldb).

 
   static bool ShouldPrint(const Function& function);