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

Side by Side Diff: tools/tickprocessor.js

Issue 2753543006: [profiler] Web UI: add summary of opts/deopts. (Closed)
Patch Set: Address reviewer comments 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 unified diff | Download patch
« no previous file with comments | « tools/profview/profview.js ('k') | no next file » | no next file with comments »
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 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
(...skipping 70 matching lines...) Expand 10 before | Expand all | Expand 10 after
81 timedRange, 81 timedRange,
82 pairwiseTimedRange, 82 pairwiseTimedRange,
83 onlySummary, 83 onlySummary,
84 runtimeTimerFilter, 84 runtimeTimerFilter,
85 preprocessJson) { 85 preprocessJson) {
86 this.preprocessJson = preprocessJson; 86 this.preprocessJson = preprocessJson;
87 LogReader.call(this, { 87 LogReader.call(this, {
88 'shared-library': { parsers: [null, parseInt, parseInt, parseInt], 88 'shared-library': { parsers: [null, parseInt, parseInt, parseInt],
89 processor: this.processSharedLibrary }, 89 processor: this.processSharedLibrary },
90 'code-creation': { 90 'code-creation': {
91 parsers: [null, parseInt, parseInt, parseInt, null, 'var-args'], 91 parsers: [null, parseInt, parseInt, parseInt, parseInt,
92 null, 'var-args'],
92 processor: this.processCodeCreation }, 93 processor: this.processCodeCreation },
93 'code-move': { parsers: [parseInt, parseInt], 94 'code-deopt': {
95 parsers: [parseInt, parseInt, parseInt, parseInt, parseInt,
96 null, null, null],
97 processor: this.processCodeDeopt },
98 'code-move': { parsers: [parseInt, parseInt, ],
94 processor: this.processCodeMove }, 99 processor: this.processCodeMove },
95 'code-delete': { parsers: [parseInt], 100 'code-delete': { parsers: [parseInt],
96 processor: this.processCodeDelete }, 101 processor: this.processCodeDelete },
97 'sfi-move': { parsers: [parseInt, parseInt], 102 'sfi-move': { parsers: [parseInt, parseInt],
98 processor: this.processFunctionMove }, 103 processor: this.processFunctionMove },
99 'active-runtime-timer': { 104 'active-runtime-timer': {
100 parsers: [null], 105 parsers: [null],
101 processor: this.processRuntimeTimerEvent }, 106 processor: this.processRuntimeTimerEvent },
102 'tick': { 107 'tick': {
103 parsers: [parseInt, parseInt, parseInt, 108 parsers: [parseInt, parseInt, parseInt,
(...skipping 155 matching lines...) Expand 10 before | Expand all | Expand 10 after
259 var self = this; 264 var self = this;
260 var libFuncs = this.cppEntriesProvider_.parseVmSymbols( 265 var libFuncs = this.cppEntriesProvider_.parseVmSymbols(
261 name, startAddr, endAddr, aslrSlide, function(fName, fStart, fEnd) { 266 name, startAddr, endAddr, aslrSlide, function(fName, fStart, fEnd) {
262 self.profile_.addStaticCode(fName, fStart, fEnd); 267 self.profile_.addStaticCode(fName, fStart, fEnd);
263 self.setCodeType(fName, 'CPP'); 268 self.setCodeType(fName, 'CPP');
264 }); 269 });
265 }; 270 };
266 271
267 272
268 TickProcessor.prototype.processCodeCreation = function( 273 TickProcessor.prototype.processCodeCreation = function(
269 type, kind, start, size, name, maybe_func) { 274 type, kind, timestamp, start, size, name, maybe_func) {
270 name = this.deserializedEntriesNames_[start] || name; 275 name = this.deserializedEntriesNames_[start] || name;
271 if (maybe_func.length) { 276 if (maybe_func.length) {
272 var funcAddr = parseInt(maybe_func[0]); 277 var funcAddr = parseInt(maybe_func[0]);
273 var state = parseState(maybe_func[1]); 278 var state = parseState(maybe_func[1]);
274 this.profile_.addFuncCode(type, name, start, size, funcAddr, state); 279 this.profile_.addFuncCode(type, name, timestamp, start, size, funcAddr, stat e);
275 } else { 280 } else {
276 this.profile_.addCode(type, name, start, size); 281 this.profile_.addCode(type, name, timestamp, start, size);
277 } 282 }
278 }; 283 };
279 284
280 285
286 TickProcessor.prototype.processCodeDeopt = function(
287 timestamp, size, code, inliningId, scriptOffset, bailoutType,
288 sourcePositionText, deoptReasonText) {
289 this.profile_.deoptCode(timestamp, code, inliningId, scriptOffset,
290 bailoutType, sourcePositionText, deoptReasonText);
291 };
292
293
281 TickProcessor.prototype.processCodeMove = function(from, to) { 294 TickProcessor.prototype.processCodeMove = function(from, to) {
282 this.profile_.moveCode(from, to); 295 this.profile_.moveCode(from, to);
283 }; 296 };
284 297
285 298
286 TickProcessor.prototype.processCodeDelete = function(start) { 299 TickProcessor.prototype.processCodeDelete = function(start) {
287 this.profile_.deleteCode(start); 300 this.profile_.deleteCode(start);
288 }; 301 };
289 302
290 303
(...skipping 626 matching lines...) Expand 10 before | Expand all | Expand 10 after
917 for (var synArg in this.argsDispatch_) { 930 for (var synArg in this.argsDispatch_) {
918 if (arg !== synArg && dispatch === this.argsDispatch_[synArg]) { 931 if (arg !== synArg && dispatch === this.argsDispatch_[synArg]) {
919 synonyms.push(synArg); 932 synonyms.push(synArg);
920 delete this.argsDispatch_[synArg]; 933 delete this.argsDispatch_[synArg];
921 } 934 }
922 } 935 }
923 print(' ' + padRight(synonyms.join(', '), 20) + " " + dispatch[2]); 936 print(' ' + padRight(synonyms.join(', '), 20) + " " + dispatch[2]);
924 } 937 }
925 quit(2); 938 quit(2);
926 }; 939 };
OLDNEW
« no previous file with comments | « tools/profview/profview.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698