Eliminate SetObjectElement calls on Treeherder

RESOLVED WORKSFORME

Status

()

Core
JavaScript Engine: JIT
RESOLVED WORKSFORME
3 years ago
a year ago

People

(Reporter: jandem, Assigned: jandem)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

3 years ago
I was profiling a Treeherder page load yesterday and noticed we spend about 8% in SetObjectElement. Angular has an apparently very hot "extend" function, that does:

    for (var keys = Object.keys(obj), j = 0, jj = keys.length; jj > j; j++) {
        var key = keys[j];
        dst[key] = obj[key]
    }

The first problem is that IonBuilder doesn't use a SETELEM cache if the access isn't dense or typed array; we probably need a new SetElement stub to handle this case.
(Assignee)

Updated

3 years ago
Component: JavaScript Engine → JavaScript Engine: JIT
(Assignee)

Updated

2 years ago
Depends on: 1209118
(Assignee)

Updated

2 years ago
Depends on: 1214126
(Assignee)

Comment 1

a year ago
The issue in comment 0 was fixed by bug 1214126, I'll just close this.
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.