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

Side by Side 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, 8 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 unified diff | Download patch
« no previous file with comments | « no previous file | src/wasm/wasm-module.cc » ('j') | src/wasm/wasm-module.cc » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef V8_COUNTERS_H_ 5 #ifndef V8_COUNTERS_H_
6 #define V8_COUNTERS_H_ 6 #define V8_COUNTERS_H_
7 7
8 #include "include/v8.h" 8 #include "include/v8.h"
9 #include "src/allocation.h" 9 #include "src/allocation.h"
10 #include "src/base/atomic-utils.h" 10 #include "src/base/atomic-utils.h"
(...skipping 937 matching lines...) Expand 10 before | Expand all | Expand 10 after
948 HT(compile_eval, V8.CompileEvalMicroSeconds, 1000000, MICROSECOND) \ 948 HT(compile_eval, V8.CompileEvalMicroSeconds, 1000000, MICROSECOND) \
949 /* Serialization as part of compilation (code caching) */ \ 949 /* Serialization as part of compilation (code caching) */ \
950 HT(compile_serialize, V8.CompileSerializeMicroSeconds, 100000, MICROSECOND) \ 950 HT(compile_serialize, V8.CompileSerializeMicroSeconds, 100000, MICROSECOND) \
951 HT(compile_deserialize, V8.CompileDeserializeMicroSeconds, 1000000, \ 951 HT(compile_deserialize, V8.CompileDeserializeMicroSeconds, 1000000, \
952 MICROSECOND) \ 952 MICROSECOND) \
953 /* Total compilation time incl. caching/parsing */ \ 953 /* Total compilation time incl. caching/parsing */ \
954 HT(compile_script, V8.CompileScriptMicroSeconds, 1000000, MICROSECOND) \ 954 HT(compile_script, V8.CompileScriptMicroSeconds, 1000000, MICROSECOND) \
955 /* Total JavaScript execution time (including callbacks and runtime calls */ \ 955 /* Total JavaScript execution time (including callbacks and runtime calls */ \
956 HT(execute, V8.Execute, 1000000, MICROSECOND) \ 956 HT(execute, V8.Execute, 1000000, MICROSECOND) \
957 /* Asm/Wasm */ \ 957 /* Asm/Wasm */ \
958 HT(wasm_instantiate_module_time, V8.WasmInstantiateModuleMicroSeconds, \ 958 /* TODO(karlschimpf) Update chrome flags to reflect asm/wasm split. */ \
959 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
960 1000000, MICROSECOND) \
961 HT(wasm_instantiate_wasm_module_time, V8.WasmInstantiateModuleMicroSeconds, \
959 1000000, MICROSECOND) \ 962 1000000, MICROSECOND) \
960 HT(wasm_decode_module_time, V8.WasmDecodeModuleMicroSeconds, 1000000, \ 963 HT(wasm_decode_module_time, V8.WasmDecodeModuleMicroSeconds, 1000000, \
961 MICROSECOND) \ 964 MICROSECOND) \
962 HT(wasm_decode_function_time, V8.WasmDecodeFunctionMicroSeconds, 1000000, \ 965 HT(wasm_decode_function_time, V8.WasmDecodeFunctionMicroSeconds, 1000000, \
963 MICROSECOND) \ 966 MICROSECOND) \
964 HT(wasm_compile_module_time, V8.WasmCompileModuleMicroSeconds, 1000000, \ 967 HT(wasm_compile_module_time, V8.WasmCompileModuleMicroSeconds, 1000000, \
965 MICROSECOND) \ 968 MICROSECOND) \
966 HT(wasm_compile_function_time, V8.WasmCompileFunctionMicroSeconds, 1000000, \ 969 HT(wasm_compile_function_time, V8.WasmCompileFunctionMicroSeconds, 1000000, \
967 MICROSECOND) \ 970 MICROSECOND) \
968 HT(asm_wasm_translation_time, V8.AsmWasmTranslationMicroSeconds, 1000000, \ 971 HT(asm_wasm_translation_time, V8.AsmWasmTranslationMicroSeconds, 1000000, \
(...skipping 348 matching lines...) Expand 10 before | Expand all | Expand 10 after
1317 1320
1318 explicit Counters(Isolate* isolate); 1321 explicit Counters(Isolate* isolate);
1319 1322
1320 DISALLOW_IMPLICIT_CONSTRUCTORS(Counters); 1323 DISALLOW_IMPLICIT_CONSTRUCTORS(Counters);
1321 }; 1324 };
1322 1325
1323 } // namespace internal 1326 } // namespace internal
1324 } // namespace v8 1327 } // namespace v8
1325 1328
1326 #endif // V8_COUNTERS_H_ 1329 #endif // V8_COUNTERS_H_
OLDNEW
« 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