Closed
Bug 1828541
Opened 1 year ago
Closed 1 year ago
Ensure we respect Ion's max script size setting when inlining in WarpOracle.cpp
Categories
(Core :: JavaScript Engine: JIT, task, P1)
Core
JavaScript Engine: JIT
Tracking
()
RESOLVED
FIXED
114 Branch
Tracking | Status | |
---|---|---|
firefox-esr102 | --- | unaffected |
firefox112 | --- | unaffected |
firefox113 | --- | fixed |
firefox114 | --- | fixed |
People
(Reporter: alexical, Assigned: alexical)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
48 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
|
Details | Review |
No description provided.
Assignee | ||
Comment 1•1 year ago
|
||
Updated•1 year ago
|
Assignee: nobody → dothayer
Status: NEW → ASSIGNED
Updated•1 year ago
|
Updated•1 year ago
|
Priority: P2 → P1
Updated•1 year ago
|
status-firefox112:
--- → unaffected
status-firefox113:
--- → affected
status-firefox114:
--- → affected
status-firefox-esr102:
--- → unaffected
Depends on: 1819722
Pushed by dothayer@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/009b852b798d Ensure we respect Ion's max script size setting r=iain
Comment 3•1 year ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 114 Branch
Comment 4•1 year ago
|
||
Did you want to nominate this for Beta uplift, Doug?
Flags: needinfo?(dothayer)
Assignee | ||
Comment 5•1 year ago
|
||
Comment on attachment 9328945 [details]
Bug 1828541 - Ensure we respect Ion's max script size setting r?iain
Beta/Release Uplift Approval Request
- User impact if declined: Some scripts could cause Ion compile times and memory usage to explode a bit, most likely just negatively affecting performance. Unsure how common these scripts actually are, but likely fairly rare.
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): It's a very straightforward patch just limiting our ability to inline scripts in Ion if too much inlining has been happening already.
- String changes made/needed: None
- Is Android affected?: Yes
Flags: needinfo?(dothayer)
Attachment #9328945 -
Flags: approval-mozilla-beta?
Comment 6•1 year ago
|
||
Comment on attachment 9328945 [details]
Bug 1828541 - Ensure we respect Ion's max script size setting r?iain
Approved for 113.0b8.
Attachment #9328945 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 7•1 year ago
|
||
bugherder uplift |
You need to log in
before you can comment on or make changes to this bug.
Description
•