Closed
Bug 1730034
Opened 3 years ago
Closed 3 years ago
TrialInlining spew should indicate polymorphism skip reason
Categories
(Core :: JavaScript Engine: JIT, enhancement, P3)
Core
JavaScript Engine: JIT
Tracking
()
RESOLVED
FIXED
94 Branch
| Tracking | Status | |
|---|---|---|
| firefox94 | --- | fixed |
People
(Reporter: tcampbell, Assigned: tcampbell)
Details
Attachments
(2 files)
The JitSpew_WarpTrialInlining lists reasons why inlining is not applied, but currently only spews if there is a specific callee. Extend this to indicate polymorphism as a failure reason.
|
Assignee | |
Comment 1•3 years ago
|
||
When doing trial inlining, show a spew message for cases that are not being
inlined due to polymorphism. Also fix indentation of SUCCESS in existing spew.
|
|
Assignee | |
Comment 2•3 years ago
|
||
Report the pcOffset on the "candidate" line, so that it is available for all
results.
Depends on D125133
Pushed by tcampbell@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/2afe50145196 Show polymorphism in WarpTrialInlining spew. r=iain https://hg.mozilla.org/integration/autoland/rev/1dca768db312 Cleanup reporting of pcOffset in WarpTrialInlining spew. r=iain
|
||
Comment 4•3 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/2afe50145196
https://hg.mozilla.org/mozilla-central/rev/1dca768db312
Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox94:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 94 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•