Linux64 devedition builds should use 3-tier PGO
Categories
(Firefox Build System :: Task Configuration, task)
Tracking
(firefox68 fixed)
Tracking | Status | |
---|---|---|
firefox68 | --- | fixed |
People
(Reporter: mshal, Assigned: mshal)
Details
Attachments
(2 files)
Comment 1•6 years ago
|
||
More generally, shouldn't the devedition builds be derived from the shippable ones?
Comment 2•6 years ago
|
||
(In reply to Mike Hommey [:glandium] from comment #1)
More generally, shouldn't the devedition builds be derived from the shippable ones?
As I said in Bug 1516114 comment 32, they should definitely be renamed shippable. I don't know enough about the differences in the mozconfigs to know whether any of them affect the compiled code. If they don't affect the compile code, it seems like it would be reasonable to change them to somehow repack the shippable build. However, that seems like it is probably enough work to be beyond the scope of this bug.
Comment 3•6 years ago
|
||
I think we have too many build types that are similar to each other and that is a footgun (change one, forget the others). We should create the devedition tasks by duplicating and tweaking the shippable tasks programmatically, or something along those lines. BTW, IIRC, there's been a bug on file for this for a long time.
Comment 4•6 years ago
|
||
(In reply to Mike Hommey [:glandium] from comment #3)
I think we have too many build types that are similar to each other and that is a footgun (change one, forget the others). We should create the devedition tasks by duplicating and tweaking the shippable tasks programmatically, or something along those lines.
I agree ... but we should make devedition builds 3-tier pgo in the meantime.
Assignee | ||
Comment 5•6 years ago
|
||
Some groups of tasks need to share the same profile data. For example,
Android PGO builds and Android Nightly builds both use the
generate-profile-android-api-16/pgo task for profile data. Previously
this was done with a text substitution, but this is a bit hacky and
doesn't easily scale with different build types.
Allowing use_pgo to be a string means we can just directly point to the
generate-profile task that contains the profile data to be used in a PGO
build.
Assignee | ||
Comment 6•6 years ago
|
||
Depends on D29247
Assignee | ||
Comment 7•6 years ago
|
||
Comment 9•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/b7aa4980e921
https://hg.mozilla.org/mozilla-central/rev/2de8fcb8a341
Description
•