Design Doc: https://docs.google.com/a/chromium.org/document/d/1eZegutNH2on-Y30hO0yKlapgfuEvPgXwFlGx7J2U5i8/edit?usp=sharing
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/949734f73adf824a5290e15a985e653c2246abb6 commit 949734f73adf824a5290e15a985e653c2246abb6 Author: Camillo Bruni <cbruni@chromium.org> Date: Mon Oct 30 15:58:48 2017 [log] Support logging basic function events This CL contains the base implementation for logging function events. Currently only compiler events are support (compile, compile-lazy...), future CLs will enable log events for parsing and first-time exeuction of functions. Bug: chromium:757467 Change-Id: Ia705979190a3ebc1009989610483a7a141bc504b Reviewed-on: https://chromium-review.googlesource.com/743921 Reviewed-by: Igor Sheludko <ishell@chromium.org> Commit-Queue: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/master@{#49040} [modify] https://crrev.com/949734f73adf824a5290e15a985e653c2246abb6/src/compiler.cc [modify] https://crrev.com/949734f73adf824a5290e15a985e653c2246abb6/src/compiler.h [modify] https://crrev.com/949734f73adf824a5290e15a985e653c2246abb6/src/flag-definitions.h [modify] https://crrev.com/949734f73adf824a5290e15a985e653c2246abb6/src/log-utils.cc [modify] https://crrev.com/949734f73adf824a5290e15a985e653c2246abb6/src/log-utils.h [modify] https://crrev.com/949734f73adf824a5290e15a985e653c2246abb6/src/log.cc [modify] https://crrev.com/949734f73adf824a5290e15a985e653c2246abb6/src/log.h [modify] https://crrev.com/949734f73adf824a5290e15a985e653c2246abb6/test/cctest/test-log.cc
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/e634740bf9c88e4208801d271d06fd70b7a6cc61 commit e634740bf9c88e4208801d271d06fd70b7a6cc61 Author: Camillo Bruni <cbruni@chromium.org> Date: Tue Nov 14 10:36:58 2017 [log] Support --log-function-events in the parser This partially retires --trace-parse and --trace-preparse which will be fully removed in a later CL. Drive-by-fix: make the Parser constructor arguments order more consistent. Bug: chromium:757467 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_chromium_rel_ng Change-Id: I80a688ce553adfa65ad346718604bd53a9e606a9 Reviewed-on: https://chromium-review.googlesource.com/744046 Commit-Queue: Camillo Bruni <cbruni@chromium.org> Reviewed-by: Marja Hölttä <marja@chromium.org> Cr-Commit-Position: refs/heads/master@{#49350} [modify] https://crrev.com/e634740bf9c88e4208801d271d06fd70b7a6cc61/src/compiler.cc [modify] https://crrev.com/e634740bf9c88e4208801d271d06fd70b7a6cc61/src/log-utils.cc [modify] https://crrev.com/e634740bf9c88e4208801d271d06fd70b7a6cc61/src/log-utils.h [modify] https://crrev.com/e634740bf9c88e4208801d271d06fd70b7a6cc61/src/log.cc [modify] https://crrev.com/e634740bf9c88e4208801d271d06fd70b7a6cc61/src/log.h [modify] https://crrev.com/e634740bf9c88e4208801d271d06fd70b7a6cc61/src/parsing/parse-info.cc [modify] https://crrev.com/e634740bf9c88e4208801d271d06fd70b7a6cc61/src/parsing/parse-info.h [modify] https://crrev.com/e634740bf9c88e4208801d271d06fd70b7a6cc61/src/parsing/parser-base.h [modify] https://crrev.com/e634740bf9c88e4208801d271d06fd70b7a6cc61/src/parsing/parser.cc [modify] https://crrev.com/e634740bf9c88e4208801d271d06fd70b7a6cc61/src/parsing/parser.h [modify] https://crrev.com/e634740bf9c88e4208801d271d06fd70b7a6cc61/src/parsing/preparser.cc [modify] https://crrev.com/e634740bf9c88e4208801d271d06fd70b7a6cc61/src/parsing/preparser.h [modify] https://crrev.com/e634740bf9c88e4208801d271d06fd70b7a6cc61/test/cctest/test-log.cc [modify] https://crrev.com/e634740bf9c88e4208801d271d06fd70b7a6cc61/test/cctest/test-parsing.cc
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/1385b092e832e2f3fcd5f27736f2dcb0cdc08a1a commit 1385b092e832e2f3fcd5f27736f2dcb0cdc08a1a Author: Camillo Bruni <cbruni@chromium.org> Date: Thu Nov 30 16:38:59 2017 [log] Support first function execution logging with --log-function-events Bug: chromium:757467 Change-Id: I52f8100e0c8b4ac07e1f875e44cb468fa8ce572b Reviewed-on: https://chromium-review.googlesource.com/793611 Reviewed-by: Ross McIlroy <rmcilroy@chromium.org> Reviewed-by: Leszek Swirski <leszeks@chromium.org> Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/master@{#49764} [modify] https://crrev.com/1385b092e832e2f3fcd5f27736f2dcb0cdc08a1a/src/builtins/arm/builtins-arm.cc [modify] https://crrev.com/1385b092e832e2f3fcd5f27736f2dcb0cdc08a1a/src/builtins/arm64/builtins-arm64.cc [modify] https://crrev.com/1385b092e832e2f3fcd5f27736f2dcb0cdc08a1a/src/builtins/ia32/builtins-ia32.cc [modify] https://crrev.com/1385b092e832e2f3fcd5f27736f2dcb0cdc08a1a/src/builtins/mips/builtins-mips.cc [modify] https://crrev.com/1385b092e832e2f3fcd5f27736f2dcb0cdc08a1a/src/builtins/mips64/builtins-mips64.cc [modify] https://crrev.com/1385b092e832e2f3fcd5f27736f2dcb0cdc08a1a/src/builtins/ppc/builtins-ppc.cc [modify] https://crrev.com/1385b092e832e2f3fcd5f27736f2dcb0cdc08a1a/src/builtins/s390/builtins-s390.cc [modify] https://crrev.com/1385b092e832e2f3fcd5f27736f2dcb0cdc08a1a/src/builtins/x64/builtins-x64.cc [modify] https://crrev.com/1385b092e832e2f3fcd5f27736f2dcb0cdc08a1a/src/compiler.cc [modify] https://crrev.com/1385b092e832e2f3fcd5f27736f2dcb0cdc08a1a/src/feedback-vector-inl.h [modify] https://crrev.com/1385b092e832e2f3fcd5f27736f2dcb0cdc08a1a/src/feedback-vector.cc [modify] https://crrev.com/1385b092e832e2f3fcd5f27736f2dcb0cdc08a1a/src/feedback-vector.h [modify] https://crrev.com/1385b092e832e2f3fcd5f27736f2dcb0cdc08a1a/src/globals.h [modify] https://crrev.com/1385b092e832e2f3fcd5f27736f2dcb0cdc08a1a/src/heap/heap.cc [modify] https://crrev.com/1385b092e832e2f3fcd5f27736f2dcb0cdc08a1a/src/objects-inl.h [modify] https://crrev.com/1385b092e832e2f3fcd5f27736f2dcb0cdc08a1a/src/objects-printer.cc [modify] https://crrev.com/1385b092e832e2f3fcd5f27736f2dcb0cdc08a1a/src/runtime/runtime-compiler.cc [modify] https://crrev.com/1385b092e832e2f3fcd5f27736f2dcb0cdc08a1a/src/runtime/runtime.h [modify] https://crrev.com/1385b092e832e2f3fcd5f27736f2dcb0cdc08a1a/test/cctest/test-log.cc
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/b0e2074d9e71150e7b4790331cab934d13c11c38 commit b0e2074d9e71150e7b4790331cab934d13c11c38 Author: Camillo Bruni <cbruni@chromium.org> Date: Wed Dec 20 13:07:17 2017 [tools] New parse processor tool From the log data generated with --log-function-events we can create a detailed model of a function's lifetime. The parse processor displays several stats at a given time (percent and count) on a per function or byte basis: - preparsing - parsing - eager/lazy compiling - execution Bug: chromium:757467 Change-Id: I0ad5c9369c6a0628704e3caffb3920444ea603a9 Reviewed-on: https://chromium-review.googlesource.com/758641 Commit-Queue: Camillo Bruni <cbruni@chromium.org> Reviewed-by: Mathias Bynens <mathias@chromium.org> Cr-Commit-Position: refs/heads/master@{#50234} [add] https://crrev.com/b0e2074d9e71150e7b4790331cab934d13c11c38/tools/parse-processor [add] https://crrev.com/b0e2074d9e71150e7b4790331cab934d13c11c38/tools/parse-processor-driver.js [add] https://crrev.com/b0e2074d9e71150e7b4790331cab934d13c11c38/tools/parse-processor.html [add] https://crrev.com/b0e2074d9e71150e7b4790331cab934d13c11c38/tools/parse-processor.js
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/b8f9b047410019887457d57adad7eb4097eb8f7d commit b8f9b047410019887457d57adad7eb4097eb8f7d Author: Camillo Bruni <cbruni@chromium.org> Date: Wed Jun 06 11:42:51 2018 [log] Fix function event logging Drive-by-fix: - improve log parsing by not wrapping single lines in arrays Change-Id: Ic4c5fdeb7875c8c5db57843f583e93285026ff74 BUG: chromium:757467, chromium:850038 Reviewed-on: https://chromium-review.googlesource.com/1078753 Commit-Queue: Camillo Bruni <cbruni@chromium.org> Reviewed-by: Jaroslav Sevcik <jarin@chromium.org> Reviewed-by: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/master@{#53547} [modify] https://crrev.com/b8f9b047410019887457d57adad7eb4097eb8f7d/tools/logreader.js [modify] https://crrev.com/b8f9b047410019887457d57adad7eb4097eb8f7d/tools/parse-processor.html [modify] https://crrev.com/b8f9b047410019887457d57adad7eb4097eb8f7d/tools/parse-processor.js
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/a76ce4f8994335c2036ae1e11ee9b85f1ca34466 commit a76ce4f8994335c2036ae1e11ee9b85f1ca34466 Author: Camillo Bruni <cbruni@chromium.org> Date: Wed Jun 20 13:40:22 2018 [log] Improve log parsing - Add parseString and parseVarArgs helper constants - Fix number formatting in parser-processor.js - Rename time to duration in parse-processor - Fix eval handling in parse-processor Bug: chromium:757467, chromium:850038 Change-Id: Ibce57b46d22e03ddaa5baa22f45d8df4c93af2cd Reviewed-on: https://chromium-review.googlesource.com/1102435 Reviewed-by: Peter Marshall <petermarshall@chromium.org> Commit-Queue: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/master@{#53887} [modify] https://crrev.com/a76ce4f8994335c2036ae1e11ee9b85f1ca34466/test/cctest/log-eq-of-logging-and-traversal.js [modify] https://crrev.com/a76ce4f8994335c2036ae1e11ee9b85f1ca34466/tools/dumpcpp.js [modify] https://crrev.com/a76ce4f8994335c2036ae1e11ee9b85f1ca34466/tools/ic-processor.js [modify] https://crrev.com/a76ce4f8994335c2036ae1e11ee9b85f1ca34466/tools/logreader.js [modify] https://crrev.com/a76ce4f8994335c2036ae1e11ee9b85f1ca34466/tools/map-processor.js [modify] https://crrev.com/a76ce4f8994335c2036ae1e11ee9b85f1ca34466/tools/parse-processor.js [modify] https://crrev.com/a76ce4f8994335c2036ae1e11ee9b85f1ca34466/tools/profviz/composer.js [modify] https://crrev.com/a76ce4f8994335c2036ae1e11ee9b85f1ca34466/tools/tickprocessor.js
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/aafd5c52ab98608e37284a5a944a4c63045625b9 commit aafd5c52ab98608e37284a5a944a4c63045625b9 Author: Camillo Bruni <cbruni@chromium.org> Date: Fri Jun 22 16:05:30 2018 [parser][log] Log script id during background compilation - Add separate script-create, script-reserve-id and script-details log events - Add log events for CompilationCache hits and puts - Simplify function event logging by only pass along the script id - Explicitly create Scripts in parse-processor.js on script events only - Create a temporary script id in the ParseInfo for use during background parsing and compilation - Clean up ParseInfo initialization to centralize creation and use of script ids - Allow creating Scripts with predefined script ids Bug: chromium:757467, chromium:850038 Change-Id: I02dfd1d5725795b9fe0ea94ef57b287b934a1efe Reviewed-on: https://chromium-review.googlesource.com/1097131 Reviewed-by: Sathya Gunasekaran <gsathya@chromium.org> Reviewed-by: Leszek Swirski <leszeks@chromium.org> Reviewed-by: Ulan Degenbaev <ulan@chromium.org> Commit-Queue: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/master@{#53978} [modify] https://crrev.com/aafd5c52ab98608e37284a5a944a4c63045625b9/src/compilation-cache.cc [modify] https://crrev.com/aafd5c52ab98608e37284a5a944a4c63045625b9/src/compilation-cache.h [modify] https://crrev.com/aafd5c52ab98608e37284a5a944a4c63045625b9/src/compiler-dispatcher/unoptimized-compile-job.cc [modify] https://crrev.com/aafd5c52ab98608e37284a5a944a4c63045625b9/src/compiler.cc [modify] https://crrev.com/aafd5c52ab98608e37284a5a944a4c63045625b9/src/heap/factory.cc [modify] https://crrev.com/aafd5c52ab98608e37284a5a944a4c63045625b9/src/heap/factory.h [modify] https://crrev.com/aafd5c52ab98608e37284a5a944a4c63045625b9/src/log.cc [modify] https://crrev.com/aafd5c52ab98608e37284a5a944a4c63045625b9/src/log.h [modify] https://crrev.com/aafd5c52ab98608e37284a5a944a4c63045625b9/src/parsing/parse-info.cc [modify] https://crrev.com/aafd5c52ab98608e37284a5a944a4c63045625b9/src/parsing/parse-info.h [modify] https://crrev.com/aafd5c52ab98608e37284a5a944a4c63045625b9/src/parsing/parser-base.h [modify] https://crrev.com/aafd5c52ab98608e37284a5a944a4c63045625b9/src/parsing/parser.cc [modify] https://crrev.com/aafd5c52ab98608e37284a5a944a4c63045625b9/src/parsing/preparser.cc [modify] https://crrev.com/aafd5c52ab98608e37284a5a944a4c63045625b9/src/runtime/runtime-compiler.cc [modify] https://crrev.com/aafd5c52ab98608e37284a5a944a4c63045625b9/test/cctest/test-log.cc [modify] https://crrev.com/aafd5c52ab98608e37284a5a944a4c63045625b9/tools/parse-processor.html [modify] https://crrev.com/aafd5c52ab98608e37284a5a944a4c63045625b9/tools/parse-processor.js
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/a800e05007d42c84cd5627e5dfb305dfb9912844 commit a800e05007d42c84cd5627e5dfb305dfb9912844 Author: Yang Guo <yangguo@chromium.org> Date: Fri Jun 22 19:46:30 2018 Revert "[parser][log] Log script id during background compilation" This reverts commit aafd5c52ab98608e37284a5a944a4c63045625b9. Reason for revert: Tentative revert for https://ci.chromium.org/p/v8/builders/luci.v8.ci/V8%20Win64/24825 https://ci.chromium.org/p/v8/builders/luci.v8.ci/V8%20Win64%20-%20msvc/3242 Original change's description: > [parser][log] Log script id during background compilation > > - Add separate script-create, script-reserve-id and script-details log events > - Add log events for CompilationCache hits and puts > - Simplify function event logging by only pass along the script id > - Explicitly create Scripts in parse-processor.js on script events only > - Create a temporary script id in the ParseInfo for use during background > parsing and compilation > - Clean up ParseInfo initialization to centralize creation and use of > script ids > - Allow creating Scripts with predefined script ids > > Bug: chromium:757467, chromium:850038 > Change-Id: I02dfd1d5725795b9fe0ea94ef57b287b934a1efe > Reviewed-on: https://chromium-review.googlesource.com/1097131 > Reviewed-by: Sathya Gunasekaran <gsathya@chromium.org> > Reviewed-by: Leszek Swirski <leszeks@chromium.org> > Reviewed-by: Ulan Degenbaev <ulan@chromium.org> > Commit-Queue: Camillo Bruni <cbruni@chromium.org> > Cr-Commit-Position: refs/heads/master@{#53978} TBR=ulan@chromium.org,cbruni@chromium.org,gsathya@chromium.org,leszeks@chromium.org Change-Id: I629f72f51d5e086e2b54658c1fdd18cec268aab2 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: chromium:757467, chromium:850038 Reviewed-on: https://chromium-review.googlesource.com/1112538 Reviewed-by: Yang Guo <yangguo@chromium.org> Commit-Queue: Yang Guo <yangguo@chromium.org> Cr-Commit-Position: refs/heads/master@{#53984} [modify] https://crrev.com/a800e05007d42c84cd5627e5dfb305dfb9912844/src/compilation-cache.cc [modify] https://crrev.com/a800e05007d42c84cd5627e5dfb305dfb9912844/src/compilation-cache.h [modify] https://crrev.com/a800e05007d42c84cd5627e5dfb305dfb9912844/src/compiler-dispatcher/unoptimized-compile-job.cc [modify] https://crrev.com/a800e05007d42c84cd5627e5dfb305dfb9912844/src/compiler.cc [modify] https://crrev.com/a800e05007d42c84cd5627e5dfb305dfb9912844/src/heap/factory.cc [modify] https://crrev.com/a800e05007d42c84cd5627e5dfb305dfb9912844/src/heap/factory.h [modify] https://crrev.com/a800e05007d42c84cd5627e5dfb305dfb9912844/src/log.cc [modify] https://crrev.com/a800e05007d42c84cd5627e5dfb305dfb9912844/src/log.h [modify] https://crrev.com/a800e05007d42c84cd5627e5dfb305dfb9912844/src/parsing/parse-info.cc [modify] https://crrev.com/a800e05007d42c84cd5627e5dfb305dfb9912844/src/parsing/parse-info.h [modify] https://crrev.com/a800e05007d42c84cd5627e5dfb305dfb9912844/src/parsing/parser-base.h [modify] https://crrev.com/a800e05007d42c84cd5627e5dfb305dfb9912844/src/parsing/parser.cc [modify] https://crrev.com/a800e05007d42c84cd5627e5dfb305dfb9912844/src/parsing/preparser.cc [modify] https://crrev.com/a800e05007d42c84cd5627e5dfb305dfb9912844/src/runtime/runtime-compiler.cc [modify] https://crrev.com/a800e05007d42c84cd5627e5dfb305dfb9912844/test/cctest/test-log.cc [modify] https://crrev.com/a800e05007d42c84cd5627e5dfb305dfb9912844/tools/parse-processor.html [modify] https://crrev.com/a800e05007d42c84cd5627e5dfb305dfb9912844/tools/parse-processor.js
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/e7f5c5ccc0a7e6b187fcc5b926546d38fe38684f commit e7f5c5ccc0a7e6b187fcc5b926546d38fe38684f Author: Camillo Bruni <cbruni@chromium.org> Date: Mon Jun 25 10:46:34 2018 Reland "[parser][log] Log script id during background compilation" This reverts commit a800e05007d42c84cd5627e5dfb305dfb9912844. Original change's description: > Revert "[parser][log] Log script id during background compilation" > > This reverts commit aafd5c52ab98608e37284a5a944a4c63045625b9. > > Reason for revert: Tentative revert for > > https://ci.chromium.org/p/v8/builders/luci.v8.ci/V8%20Win64/24825 > https://ci.chromium.org/p/v8/builders/luci.v8.ci/V8%20Win64%20-%20msvc/3242 > > Original change's description: > > [parser][log] Log script id during background compilation > > > > - Add separate script-create, script-reserve-id and script-details log events > > - Add log events for CompilationCache hits and puts > > - Simplify function event logging by only pass along the script id > > - Explicitly create Scripts in parse-processor.js on script events only > > - Create a temporary script id in the ParseInfo for use during background > > parsing and compilation > > - Clean up ParseInfo initialization to centralize creation and use of > > script ids > > - Allow creating Scripts with predefined script ids > > > > Bug: chromium:757467, chromium:850038 > > Change-Id: I02dfd1d5725795b9fe0ea94ef57b287b934a1efe > > Reviewed-on: https://chromium-review.googlesource.com/1097131 > > Reviewed-by: Sathya Gunasekaran <gsathya@chromium.org> > > Reviewed-by: Leszek Swirski <leszeks@chromium.org> > > Reviewed-by: Ulan Degenbaev <ulan@chromium.org> > > Commit-Queue: Camillo Bruni <cbruni@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#53978} > > TBR=ulan@chromium.org,cbruni@chromium.org,gsathya@chromium.org,leszeks@chromium.org > > Change-Id: I629f72f51d5e086e2b54658c1fdd18cec268aab2 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: chromium:757467, chromium:850038 > Reviewed-on: https://chromium-review.googlesource.com/1112538 > Reviewed-by: Yang Guo <yangguo@chromium.org> > Commit-Queue: Yang Guo <yangguo@chromium.org> > Cr-Commit-Position: refs/heads/master@{#53984} TBR=ulan@chromium.org,yangguo@chromium.org,cbruni@chromium.org,gsathya@chromium.org,leszeks@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: chromium:757467, chromium:850038 Change-Id: I3088c86362c06ee50464f1f14e25350b1b8048ad Reviewed-on: https://chromium-review.googlesource.com/1112539 Reviewed-by: Ulan Degenbaev <ulan@chromium.org> Reviewed-by: Yang Guo <yangguo@chromium.org> Commit-Queue: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/master@{#53994} [modify] https://crrev.com/e7f5c5ccc0a7e6b187fcc5b926546d38fe38684f/src/compilation-cache.cc [modify] https://crrev.com/e7f5c5ccc0a7e6b187fcc5b926546d38fe38684f/src/compilation-cache.h [modify] https://crrev.com/e7f5c5ccc0a7e6b187fcc5b926546d38fe38684f/src/compiler-dispatcher/unoptimized-compile-job.cc [modify] https://crrev.com/e7f5c5ccc0a7e6b187fcc5b926546d38fe38684f/src/compiler.cc [modify] https://crrev.com/e7f5c5ccc0a7e6b187fcc5b926546d38fe38684f/src/heap/factory.cc [modify] https://crrev.com/e7f5c5ccc0a7e6b187fcc5b926546d38fe38684f/src/heap/factory.h [modify] https://crrev.com/e7f5c5ccc0a7e6b187fcc5b926546d38fe38684f/src/log.cc [modify] https://crrev.com/e7f5c5ccc0a7e6b187fcc5b926546d38fe38684f/src/log.h [modify] https://crrev.com/e7f5c5ccc0a7e6b187fcc5b926546d38fe38684f/src/parsing/parse-info.cc [modify] https://crrev.com/e7f5c5ccc0a7e6b187fcc5b926546d38fe38684f/src/parsing/parse-info.h [modify] https://crrev.com/e7f5c5ccc0a7e6b187fcc5b926546d38fe38684f/src/parsing/parser-base.h [modify] https://crrev.com/e7f5c5ccc0a7e6b187fcc5b926546d38fe38684f/src/parsing/parser.cc [modify] https://crrev.com/e7f5c5ccc0a7e6b187fcc5b926546d38fe38684f/src/parsing/preparser.cc [modify] https://crrev.com/e7f5c5ccc0a7e6b187fcc5b926546d38fe38684f/src/runtime/runtime-compiler.cc [modify] https://crrev.com/e7f5c5ccc0a7e6b187fcc5b926546d38fe38684f/test/cctest/test-log.cc [modify] https://crrev.com/e7f5c5ccc0a7e6b187fcc5b926546d38fe38684f/tools/parse-processor.html [modify] https://crrev.com/e7f5c5ccc0a7e6b187fcc5b926546d38fe38684f/tools/parse-processor.js
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/fc59b4e4e01005ca26fd770190e5c99a05d4fdad commit fc59b4e4e01005ca26fd770190e5c99a05d4fdad Author: Camillo Bruni <cbruni@chromium.org> Date: Wed Jun 27 15:32:37 2018 [log][tools] Fix logging on deserialized scripts - Introduce ScriptEventType enum - Emit log event when deserializing scripts - Better handle deserialized and eval scripts when calculating the byte size Bug: chromium:757467, chromium:850038 Change-Id: I082161d8d1d4238a90d3607ca8b6b4e60f2f903e Reviewed-on: https://chromium-review.googlesource.com/1114845 Commit-Queue: Camillo Bruni <cbruni@chromium.org> Reviewed-by: Ulan Degenbaev <ulan@chromium.org> Reviewed-by: Sathya Gunasekaran <gsathya@chromium.org> Reviewed-by: Yang Guo <yangguo@chromium.org> Cr-Commit-Position: refs/heads/master@{#54061} [modify] https://crrev.com/fc59b4e4e01005ca26fd770190e5c99a05d4fdad/src/compiler.cc [modify] https://crrev.com/fc59b4e4e01005ca26fd770190e5c99a05d4fdad/src/heap/factory.cc [modify] https://crrev.com/fc59b4e4e01005ca26fd770190e5c99a05d4fdad/src/log.cc [modify] https://crrev.com/fc59b4e4e01005ca26fd770190e5c99a05d4fdad/src/log.h [modify] https://crrev.com/fc59b4e4e01005ca26fd770190e5c99a05d4fdad/src/parsing/parse-info.cc [modify] https://crrev.com/fc59b4e4e01005ca26fd770190e5c99a05d4fdad/src/snapshot/deserializer.cc [modify] https://crrev.com/fc59b4e4e01005ca26fd770190e5c99a05d4fdad/src/snapshot/object-deserializer.cc [modify] https://crrev.com/fc59b4e4e01005ca26fd770190e5c99a05d4fdad/tools/parse-processor [modify] https://crrev.com/fc59b4e4e01005ca26fd770190e5c99a05d4fdad/tools/parse-processor.js
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/a8582eb2f11076492309df0db57e651bfbeebca5 commit a8582eb2f11076492309df0db57e651bfbeebca5 Author: Camillo Bruni <cbruni@chromium.org> Date: Tue Jul 03 09:12:44 2018 [tools] parse-processor improvements - display script size overview - color scripts in overview depending on eval, streaming or other scripts - fix stats to always take own-bytes into accout - rename all *Time properties to *Duration for consistency - extract ScriptSource log event into separate method - support script source events in parse-processor Bug: chromium:757467, chromium:850038 Change-Id: I227d1d5952ae9e508ab1a01146fcf47f74a3f7ea Reviewed-on: https://chromium-review.googlesource.com/1117195 Commit-Queue: Camillo Bruni <cbruni@chromium.org> Reviewed-by: Sathya Gunasekaran <gsathya@chromium.org> Cr-Commit-Position: refs/heads/master@{#54159} [modify] https://crrev.com/a8582eb2f11076492309df0db57e651bfbeebca5/src/log.cc [modify] https://crrev.com/a8582eb2f11076492309df0db57e651bfbeebca5/src/log.h [modify] https://crrev.com/a8582eb2f11076492309df0db57e651bfbeebca5/tools/parse-processor.html [modify] https://crrev.com/a8582eb2f11076492309df0db57e651bfbeebca5/tools/parse-processor.js [modify] https://crrev.com/a8582eb2f11076492309df0db57e651bfbeebca5/tools/tickprocessor.js
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/e3b4ffa9ba3bd602ac1ef158d6466ed55be8d182 commit e3b4ffa9ba3bd602ac1ef158d6466ed55be8d182 Author: Camillo Bruni <cbruni@chromium.org> Date: Mon Jul 16 14:05:14 2018 [tools] Improve function event logging and parse processor - Log script sources with --log-function-events - Don't show confusing duration in graphs - Introduce separate compilation category - Log script details after deserialization - Log parse times for eval scripts - Display deserialized scripts and functions Change-Id: I58f3bf8efe3955632322f958716c36ad38761fd0 Bug: chromium:757467, chromium:850038 Reviewed-on: https://chromium-review.googlesource.com/1128082 Reviewed-by: Yang Guo <yangguo@chromium.org> Reviewed-by: Sathya Gunasekaran <gsathya@chromium.org> Commit-Queue: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/master@{#54465} [modify] https://crrev.com/e3b4ffa9ba3bd602ac1ef158d6466ed55be8d182/src/compiler.cc [modify] https://crrev.com/e3b4ffa9ba3bd602ac1ef158d6466ed55be8d182/src/log.cc [modify] https://crrev.com/e3b4ffa9ba3bd602ac1ef158d6466ed55be8d182/src/log.h [modify] https://crrev.com/e3b4ffa9ba3bd602ac1ef158d6466ed55be8d182/src/snapshot/code-serializer.cc [modify] https://crrev.com/e3b4ffa9ba3bd602ac1ef158d6466ed55be8d182/src/snapshot/deserializer.cc [modify] https://crrev.com/e3b4ffa9ba3bd602ac1ef158d6466ed55be8d182/src/snapshot/object-deserializer.cc [modify] https://crrev.com/e3b4ffa9ba3bd602ac1ef158d6466ed55be8d182/tools/parse-processor.html [modify] https://crrev.com/e3b4ffa9ba3bd602ac1ef158d6466ed55be8d182/tools/parse-processor.js
Comment 1 by cbruni@chromium.org
, Aug 21 2017