Closed Bug 1542133 Opened 5 years ago Closed 5 years ago

9.45 - 12.27% build times (windows2012-64-shippable) regression on push 5851c65af0e0567d9cea916dd1d303f3b6ebe83b


(Firefox Build System :: General, defect)

Not set


(Not tracked)



(Reporter: igoldan, Unassigned)



(Keywords: regression)

We have detected a build metrics regression from push:

As author of one of the patches included in that push, we need your help to address this regression.


12% build times windows2012-64-shippable opt nightly taskcluster-c4.4xlarge 5,442.46 -> 6,110.33
9% build times windows2012-64-shippable opt nightly taskcluster-c5.4xlarge 4,556.19 -> 4,986.55

You can find links to graphs and comparison views for each of the above tests at:

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:

*** Please let us know your plans within 3 business days, or the offending patch(es) will be backed out! ***

Product: Testing → Firefox Build System
Flags: needinfo?(dmajor)

It is not surprising to see a build time increase from the additional optimizations that we enabled.

As for whether the benefits are worth the cost, I defer to Anthony.

Flags: needinfo?(dmajor) → needinfo?(ajones)

FWIW this not going to affect standard developer builds unless they're doing PGO, which seems unlikely. It should also only affect "shippable" builds in automation which seems reasonable.

(In reply to David Major [:dmajor] (low availability) from comment #1)

As for whether the benefits are worth the cost, I defer to Anthony.

The increase in compilation time is expected. XLTO saves deveoper time by allowing developers to assume reasonable inlining across languages. In the past this the lack of XLTO has been worked around by generating the same code in C++ and Rust e.g. for style structs.

Flags: needinfo?(ajones)
Closed: 5 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.