Closed
Bug 1637802
Opened 6 years ago
Closed 6 years ago
Add markers for script compilation and execution
Categories
(Core :: Performance: General, task)
Core
Performance: General
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox78 | --- | fixed |
People
(Reporter: denispal, Assigned: denispal)
Details
Attachments
(1 file)
We currently already have a script evaluation marker, but we can break this up into a compilation marker for main thread and off thread, and an execution marker for a bit more granularity.
| Assignee | ||
Updated•6 years ago
|
Assignee: nobody → dpalmeiro
| Assignee | ||
Comment 1•6 years ago
|
||
Replace the existing ScriptEvaluation profiler marker with ScriptCompileMainThread, ScriptCompileOffThread, and ScriptExecution for more granularity.
| Assignee | ||
Comment 2•6 years ago
|
||
Also adding some changes so we cover bytecode and binast sources as well.
Updated•6 years ago
|
Whiteboard: [qf:p3]
Updated•6 years ago
|
Attachment #9148203 -
Attachment description: Replace ScriptEvaluation profiler marker with ScriptCompileMainThread, ScriptCompileOffThread, and ScriptExecution → Replace ScriptEvaluation profiler marker with compilation & execution markers for bytecode, module, binast, and text sources.
| Assignee | ||
Comment 3•6 years ago
|
||
Some example profiles with the new markers:
https://perfht.ml/3g7sV1d
https://perfht.ml/2ykWZWk
https://perfht.ml/3e7VSbT
| Assignee | ||
Comment 4•6 years ago
|
||
One more with the bytecode off main thread marker as well:
https://perfht.ml/3gflCEY
Pushed by dpalmeiro@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7aa26615adf9
Replace ScriptEvaluation profiler marker with compilation & execution markers for bytecode, module, binast, and text sources. r=smaug
Comment 6•6 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox78:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla78
Updated•4 years ago
|
Performance Impact: --- → P3
Whiteboard: [qf:p3]
You need to log in
before you can comment on or make changes to this bug.
Description
•