764310 - IonMonkey: Compile JSOP_DEFFUN

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Jan de Mooij [:jandem]]

We probably don't need this for the initial release since we can still fallback to JM+TI, but JSOP_DEFFUN is pretty common, especially in global scripts - see for instance SS base64.

Comment 1

[Jan de Mooij [:jandem]]

Most of the work here we will also need for the baseline compiler, so taking.

Comment 2

[Jan de Mooij [:jandem]]

Attachment: Part 1: Refactor JSOP_DEFFUN interpreter logic

Comment 3

[Jan de Mooij [:jandem]]

Attachment: Part 2: Compile DEFFUN

DEFFUN is used for function statements in the global scope, so this will also help test coverage.

Comment 4

[Jan de Mooij [:jandem]]

Attachment: Part 1: Refactor JSOP_DEFFUN interpreter logic

Comment 5

[Jan de Mooij [:jandem]]

Attachment: Part 2: Compile DEFFUN

Comment 6

[Jan de Mooij [:jandem]]

Part 1:

https://hg.mozilla.org/integration/mozilla-inbound/rev/671e04519d02

Comment 7

[Jan de Mooij [:jandem]]

Part 2:

https://hg.mozilla.org/integration/mozilla-inbound/rev/88a218a4b5bf

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/671e04519d02
https://hg.mozilla.org/mozilla-central/rev/88a218a4b5bf