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

Unified Diff: src/wasm/module-decoder.cc

Issue 2772363002: Separate function decoding counter into asm and wasm counters. (Closed)
Patch Set: Create dummy module environment for DecodeWasmFunction test. 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
Index: src/wasm/module-decoder.cc
diff --git a/src/wasm/module-decoder.cc b/src/wasm/module-decoder.cc
index a26f0047b43e2c2860811b0967b24134d172d142..84c39a9092f99d5c9a7b8d09e1541851ea8a57e6 100644
--- a/src/wasm/module-decoder.cc
+++ b/src/wasm/module-decoder.cc
@@ -1196,7 +1196,9 @@ FunctionResult DecodeWasmFunction(Isolate* isolate, Zone* zone,
const byte* function_start,
const byte* function_end) {
HistogramTimerScope wasm_decode_function_time_scope(
- isolate->counters()->wasm_decode_function_time());
+ module_env->module_env.is_wasm()
+ ? isolate->counters()->wasm_decode_wasm_function_time()
+ : isolate->counters()->wasm_decode_asm_function_time());
size_t size = function_end - function_start;
if (function_start > function_end) return FunctionError("start > end");
if (size > kV8MaxWasmFunctionSize)

Powered by Google App Engine
This is Rietveld 408576698