Separate module instantiate counter for asm and wasm.

src/counters.h

Index: src/counters.h
diff --git a/src/counters.h b/src/counters.h
index 39d97dd018b1d5f784b77fc70ca88ea831d55d71..cb139b1400513e317f094f16eabe94085cfa864e 100644
--- a/src/counters.h
+++ b/src/counters.h
@@ -955,7 +955,10 @@ class RuntimeCallTimerScope {
   /* Total JavaScript execution time (including callbacks and runtime calls */ \
   HT(execute, V8.Execute, 1000000, MICROSECOND)                                \
   /* Asm/Wasm */                                                               \
-  HT(wasm_instantiate_module_time, V8.WasmInstantiateModuleMicroSeconds,       \
+  /* TODO(karlschimpf) Update chrome flags to reflect asm/wasm split. */       \
+  HT(wasm_instantiate_asm_module_time, V8.WasmInstantiateModuleMicroSeconds,   \
+     1000000, MICROSECOND)                                                     \
+  HT(wasm_instantiate_wasm_module_time, V8.WasmInstantiateModuleMicroSeconds,  \

[bbudge, 2017/03/23 22:04:58]

It seems strange to me to break out these separate flags, but then use the same
histogram bucket. Perhaps I don't understand?

[Karl, 2017/03/24 16:49:19]

Since the V8.flags are from Chrome, I am trying to put off those changes and
update them all at once.

This way, I can split up many of these counters (based on asm/wasm origin) as
separate CLs without having to fight each one through a chrome review.

      1000000, MICROSECOND)                                                     \
   HT(wasm_decode_module_time, V8.WasmDecodeModuleMicroSeconds, 1000000,        \
      MICROSECOND)                                                              \