1308634 - Inline Object.prototype.hasOwnProperty().

Status: RESOLVED INVALID

(Core :: JavaScript Engine: JIT, defect)

Description

[Sean Stangl [:sstangl]]

The following testcase takes 0.6s in d8, 13.6s in SpiderMonkey:

> function f(o) {
>     k = true;
>     for (var i = 0; i < 10000000; i++) {
>         k = k ^ o.hasOwnProperty('p');
>     }
>     return k;
> }   
>
> f({p: 5});

Google Closure Compiler generates code that uses hasOwnProperty() to check for existence.

Comment 1

[Sean Stangl [:sstangl]]

Realized I was accidentally using a debug version. We're twice as fast as d8.

Comment 2

[theo.rabut]

Attachment: Bug 1308634 - Implicit call to SetValueAtTime in SetCurveAtTime, tests, RangeError changed to NotSupportedError

Review commit: https://reviewboard.mozilla.org/r/204818/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/204818/