| Summary: | Add an option for logging total phase times | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Saam Barati <saam> | ||||||
| Component: | JavaScriptCore | Assignee: | Saam Barati <saam> | ||||||
| Status: | RESOLVED FIXED | ||||||||
| Severity: | Normal | CC: | benjamin, commit-queue, fpizlo, ggaren, gskachkov, guijemont, keith_miller, mark.lam, msaboff, rmorisset, ticaiolima, tzagallo, webkit-bug-importer, ysuzuki | ||||||
| Priority: | P2 | Keywords: | InRadar | ||||||
| Version: | WebKit Nightly Build | ||||||||
| Hardware: | Unspecified | ||||||||
| OS: | Unspecified | ||||||||
| Attachments: |
|
||||||||
|
Description
Saam Barati
2020-01-22 15:53:01 PST
Created attachment 388490 [details]
patch
Comment on attachment 388490 [details]
patch
r=me
Comment on attachment 388490 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=388490&action=review r=me too with comment. > Source/JavaScriptCore/runtime/OptionsList.h:167 > + v(Bool, reportTotalPhaseTimes, false, Normal, nullptr) \ Plox add a string here that explains the difference between this only prints at the end. Created attachment 388492 [details]
patch for landing
Comment on attachment 388492 [details] patch for landing Clearing flags on attachment: 388492 Committed r254957: <https://trac.webkit.org/changeset/254957> All reviewed patches have been landed. Closing bug. |