Closed
Bug 1503805
Opened 6 years ago
Closed 6 years ago
359.98 - 459.8% compiler_metrics num_static_constructors (windows2012-32, windows2012-64) regression on push 0c7b395eedb4b03e90bd2b0ac909582f37f77677 (Wed Oct 24 2018)
Categories
(Testing :: General, defect)
Tracking
(Not tracked)
RESOLVED
INVALID
People
(Reporter: igoldan, Unassigned)
References
Details
(Keywords: regression)
We have detected a build metrics regression from push:
https://hg.mozilla.org/releases/mozilla-beta/pushloghtml?changeset=0c7b395eedb4b03e90bd2b0ac909582f37f77677
As author of one of the patches included in that push, we need your help to address this regression.
Regressions:
460% compiler_metrics num_static_constructors windows2012-64 opt msvc 101.29 -> 567.00
360% compiler_metrics num_static_constructors windows2012-32 opt msvc 101.31 -> 466.00
You can find links to graphs and comparison views for each of the above tests at: https://treeherder.mozilla.org/perf.html#/alerts?id=17195
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 | ||
Comment 1•6 years ago
|
||
I tried bisecting this on Try, but didn't manage to find the culprit. So, please state which of the following bugs is more related to these regressions?
bug 1498710
bug 1374790
bug 1499996
bug 1500475
bug 1497860
bug 1497575
bug 1459891
Flags: needinfo?(sfraser)
Flags: needinfo?(nchevobbe)
Flags: needinfo?(mozilla)
Flags: needinfo?(masayuki)
Flags: needinfo?(kats)
Flags: needinfo?(dtownsend)
Comment 2•6 years ago
|
||
None of those pushes touches any C++ files so my guess is that Tom's taskcluster changes didn't go quite right and changed compiler options or something.
Flags: needinfo?(kats)
Comment 3•6 years ago
|
||
I think the public suffix list one (bug 1459891) it one of the least likely on the list, so am clearing needinfo from me
Flags: needinfo?(sfraser)
Comment 4•6 years ago
|
||
I just added a line into a mochitest so that must not be caused by bug1499996.
Flags: needinfo?(masayuki)
Comment 6•6 years ago
|
||
This is an expected change. The msvc builds on beta were not using the correct mozconfig, so were being compiled with clang instead. The metrics here are in line with the msvc metrics on mozilla-central.
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: needinfo?(nchevobbe)
Flags: needinfo?(mozilla)
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•