Closed
Bug 1378052
Opened 7 years ago
Closed 7 years ago
7.28% build times (linux64) regression on push ef52b30f64606d6e0c2f2781b37812f8bb5ae179 (Sat Jul 1 2017)
Categories
(Firefox Build System :: General, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: igoldan, Unassigned)
References
Details
(Keywords: regression)
We have detected a build metrics regression from push:
https://hg.mozilla.org/integration/autoland/pushloghtml?changeset=ef52b30f64606d6e0c2f2781b37812f8bb5ae179
As author of one of the patches included in that push, we need your help to address this regression.
Regressions:
7% build times summary linux64 asan asan-fuzzing taskcluster-c4.4xlarge 1,198.96 -> 1,286.27
You can find links to graphs and comparison views for each of the above tests at: https://treeherder.mozilla.org/perf.html#/alerts?id=7652
On the page above you can see an alert for each affected platform as well as a link to a graph showing the history of scores for this test. There is also a link to a treeherder page showing the jobs in a pushlog format.
To learn more about the regressing test(s), please see: https://developer.mozilla.org/en-US/docs/Mozilla/Performance/Automated_Performance_Testing_and_Sheriffing/Build_Metrics
Reporter | ||
Updated•7 years ago
|
Reporter | ||
Comment 1•7 years ago
|
||
:rforbes Could you take a look over this build time regression and estimate a fix time for it?
Flags: needinfo?(rforbes)
Comment 2•7 years ago
|
||
:rforbes is on PTO.
In bug 1376978, we switched the compiler from Clang 3.9 to Clang 4.0. It is well possible that this increases the build time, though I don't see what we could do about that. The changeset does nothing else besides the compiler switch.
Flags: needinfo?(ionut.goldan)
Comment 3•7 years ago
|
||
There's not much we can do here except profile compiler invocations and report performance regressions upstream. Sometimes compilers get faster. Sometimes they get slower. Often when they get slower it is for a good reason.
FWIW, bug 1376968 decreased the compile time substantially, offsetting this loss.
Reporter | ||
Comment 4•7 years ago
|
||
Thanks for the feedback. This is reasonable to accept, so I'm marking it as WONTFIX.
Flags: needinfo?(rforbes)
Flags: needinfo?(ionut.goldan)
Reporter | ||
Updated•7 years ago
|
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•