826647 - BaselineCompiler: Compile constructors

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Jan de Mooij [:jandem]]

Attachment: Patch

Just like Ion, when entering from the interpreter and are constructing, we have to create the object.

Comment 1

[Kannan Vijayan [:djvj]]

Comment on attachment 697858 [details] [diff] [review]
Patch

Review of attachment 697858 [details] [diff] [review]:
-----------------------------------------------------------------

::: js/src/ion/shared/IonFrames-x86-shared.h
@@ +79,5 @@
>      }
>  
> +    Value thisv() {
> +        return argv()[0];
> +    }

This should be added to IonFrames-arm.h as well.

Comment 2

[Jan de Mooij [:jandem]]

https://hg.mozilla.org/projects/ionmonkey/rev/618c1544ac42

(In reply to Kannan Vijayan [:djvj] from comment #1)
> This should be added to IonFrames-arm.h as well.

Oops, good catch.