Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1547)

Unified Diff: src/counters.h

Issue 2772773004: Separate module instantiate counter for asm and wasm. (Closed)
Patch Set: Simplify code back to previous form Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | src/wasm/wasm-module.cc » ('j') | src/wasm/wasm-module.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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, \
bradnelson 2017/03/24 17:25:10 You need to make up separate names for the chrome
bradnelson 2017/03/24 17:33:14 Sounds like these are safe to direct to the same b
+ 1000000, MICROSECOND) \
+ HT(wasm_instantiate_wasm_module_time, V8.WasmInstantiateModuleMicroSeconds, \
1000000, MICROSECOND) \
HT(wasm_decode_module_time, V8.WasmDecodeModuleMicroSeconds, 1000000, \
MICROSECOND) \
« no previous file with comments | « no previous file | src/wasm/wasm-module.cc » ('j') | src/wasm/wasm-module.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698