Closed
Bug 794117
Opened 12 years ago
Closed 12 years ago
IonMonkey: string.split, array.map and array.filter slower with Ion
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
People
(Reporter: jandem, Unassigned)
References
(Blocks 1 open bug)
Details
(Whiteboard: [ion:t])
cers reported this on IRC. For the "split" micro-benchmark below we have:
JM+TI : 1198 ms
Ion : 2594 ms
The problem is that these functions call GetTypeCallerInitObject, which calls currentScript. We want to self-host map and filter eventually, and we could specialize split.
function f() {
var s = "";
var t = new Date;
for (var i=0; i<10000000; i++) {
s.split(s);
}
print(new Date - t);
}
f();
Comment 1•12 years ago
|
||
The real issue is that we end up in GetPcScript. See bug 728045.
Bleh - once again, I think this is indicative of a design flaw in the engine, and not a problem with GetPcScript. Though if there's no easy workaround for these maybe we should hack something up, like a one-entry cache.
Updated•12 years ago
|
Whiteboard: [ion:t]
Comment 3•12 years ago
|
||
Well, by "real issue" I meant "where the time gets spent". No opinion on how to fix it. ;)
Comment 4•12 years ago
|
||
With Bug 794679 fixed, local results are 1488 ms with --no-ion, 1579 with ion.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 5•12 years ago
|
||
Well, it's still slower, just not as much slower, right?
You need to log in
before you can comment on or make changes to this bug.
Description
•