If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

11% regression on dromaeo-object-array on May 30th

RESOLVED INCOMPLETE

Status

()

Core
JavaScript Engine: JIT
RESOLVED INCOMPLETE
2 years ago
a year ago

People

(Reporter: h4writer, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

2 years ago
A regression/improvement was reported on AWFY:
- slave: Win 8 32-bit (i7-4700HQ, browser)
- mode: Ion

Regression(s)/Improvement(s):
- dromaeo: dromaeo-object-array: -11.03% (regression)

More details: http://arewefastyet.com/regressions/#/regression/1697289

I suspect:
Brian Hackett — Bug 1166678 - Optimize Array.prototype.slice in Ion, r=jandem.
https://hg.mozilla.org/integration/mozilla-inbound/rev/57bd26cc44b9

Looking at the source of dromaeo-object-array this seems plausible:
view-source:http://dromaeo.com/tests/dromaeo-object-array.html


>	test("Array Deconstruction, shift", function(){
>		var a = ret.slice();
>		for ( var j = 0; j < i; j++ )
>			tmp = a.shift();
>	});
>
>	test("Array Deconstruction, splice", function(){
>		var a = ret.slice();
>		for ( var j = 0; j < i; j++ )
>			tmp = a.splice(0,1);
>	});
>
>	test("Array Construction, push", function(){
>		ret = [];
>		for ( var j = 0; j < i * 25; j++ )
>			ret.push(j);
>	});
>
>	test("Array Deconstruction, pop", function(){
>		var a = ret.slice();
>		for ( var j = 0; j < i * 25; j++ )
>			tmp = a.pop();
>	});
(Reporter)

Comment 1

2 years ago
Could it be this optimization that was introduced, threw something off in the dromaeo-object-array case?
Can you take a look?
Flags: needinfo?(bhackett1024)
I tried running dromaeo-object-array in the shell (10.9 x86), and it seems to be maybe 5% faster (pretty noisy) when the Array.slice optimization is enabled than when it is disabled.
Flags: needinfo?(bhackett1024)
The regression is still there, but it's been very long and it's not really actionable, so closing.
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.