1416741 - 4.35 - 4.54% build times (windows2012-32, windows2012-64) regression on push ae668855bc6ecb341599a1674dddd11c90b2d62e (Sun Nov 12 2017)

Status: RESOLVED WONTFIX

(Core :: CSS Parsing and Computation, defect, P5)

Description

[Ionuț Goldan [:igoldan]]

We have detected a build metrics regression from push:

https://hg.mozilla.org/integration/autoland/pushloghtml?changeset=ae668855bc6ecb341599a1674dddd11c90b2d62e

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

Regressions:

  5%  build times summary windows2012-32 opt static-analysis taskcluster-c4.4xlarge     2,569.48 -> 2,686.21
  4%  build times summary windows2012-64 opt static-analysis taskcluster-c4.4xlarge     2,698.01 -> 2,815.25


You can find links to graphs and comparison views for each of the above tests at: https://treeherder.mozilla.org/perf.html#/alerts?id=10496

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

Comment 1

[Ionuț Goldan [:igoldan]]

:emilio The build time increased after the #19186 servo merge. Is there something we can do to reduce it?

Comment 2

[Emilio Cobos Álvarez (:emilio)]

Huh, not really, I don't have any bright idea... We're just generating more code at build time instead of writing it by hand.

Comment 3

[Sylvestre Ledru [:Sylvestre]]

Seems that it is impacting the static-analysis job. Not sure we really care about performance for this job?

Comment 4

[Emilio Cobos Álvarez (:emilio)]

I'm going to call this WONTFIX, I believe. There's no intention to write more code by hand than necessary.