Separate module compile counter for asm and wasm.

src/wasm/wasm-module.cc

Index: src/wasm/wasm-module.cc
diff --git a/src/wasm/wasm-module.cc b/src/wasm/wasm-module.cc
index 8fd2b64845376490881ec6aa79ad2483f40d2109..a03e21fc504d96d7bfed9818f20971f0188a37f5 100644
--- a/src/wasm/wasm-module.cc
+++ b/src/wasm/wasm-module.cc
@@ -514,7 +514,9 @@ class CompilationHelper {
     }
 
     HistogramTimerScope wasm_compile_module_time_scope(
-        isolate_->counters()->wasm_compile_module_time());
+        module_->origin == ModuleOrigin::kWasmOrigin

[bbudge, 2017/03/24 19:19:01]

Since we'll be doing this check all over the place, it's probably worth
pre-computing this boolean in the constructor and keeping it as an instance
field. You can add a DCHECK that it's either kWasmOrigin or kAsmJsOrigin to
prevent breakage, something like:

is_wasm_origin_ = module_->origin == ModuleOrigin::kWasmOrigin;
DCHECK_IMPLIES(!is_wasm_origin_, module_origin == ModuleOrigin::kAsmJsOrigin);

Then just use the instance field everywhere you need to switch between counters.

[Karl, 2017/03/24 21:19:58]

Part of the problem is that the origin field is NOT set in the constructor.
However, we should still use accessors like is_wasm() and is_asm_js() (as well
as get_origin() and set_origin()). This would allow us to cache a boolean
without having to change any code other than for the class definition. I'll do
that.

+            ? isolate_->counters()->wasm_compile_wasm_module_time()
+            : isolate_->counters()->wasm_compile_asm_module_time());
 
     ModuleBytesEnv module_env(module_, &temp_instance, wire_bytes);