Closed
Bug 1678485
Opened 4 years ago
Closed 4 years ago
Enable PGO on arm64 mac builds
Categories
(Firefox Build System :: Task Configuration, task)
Firefox Build System
Task Configuration
Tracking
(firefox84 fixed, firefox85 fixed)
RESOLVED
FIXED
85 Branch
People
(Reporter: glandium, Assigned: glandium)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
47 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
|
Details | Review |
No description provided.
Assignee | ||
Comment 1•4 years ago
|
||
Pushed by mh@glandium.org: https://hg.mozilla.org/integration/autoland/rev/7aa6dbc8826d Enable PGO on arm64 mac builds. r=firefox-build-system-reviewers,andi,rstewart
Assignee | ||
Comment 3•4 years ago
|
||
Comment on attachment 9189133 [details]
Bug 1678485 - Enable PGO on arm64 mac builds.
Beta/Release Uplift Approval Request
- User impact if declined: Better performance for Apple Silicon builds
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: 1678291 1677023 1677022 1677020 1678154
- Risk to taking this patch: Medium
- Why is the change risky/not risky? (and alternatives if risky): There's the possibility that this triggers some miscompilation.
But all the patches involved are no-ops for the other builds.
Also one side effect is that bug 1678154 makes the Apple Silicon builds faster and not rely on a small pool of worker. - String changes made/needed:
Attachment #9189133 -
Flags: approval-mozilla-beta?
Comment 4•4 years ago
|
||
Comment on attachment 9189133 [details]
Bug 1678485 - Enable PGO on arm64 mac builds.
This isn't normally the kind of change I'd take in a Beta, but given that it's a new platform shipping for the first time to a small group of real-world users and there's likely a lot of interest in Firefox' performance on Apple Silicon, it seems like the risk is justified here. Approved for 84.0b4.
Attachment #9189133 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 5•4 years ago
|
||
bugherder uplift |
status-firefox84:
--- → fixed
Comment 6•4 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox85:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 85 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•