1417038 - Shrink ExitFooterFrame to 1 word

Status: RESOLVED FIXED

(Core :: JavaScript Engine: JIT, enhancement)

Description

[Jan de Mooij [:jandem]]

Instead of storing both JitCode* and VMFunction* we can now store 1 pointer that's either the ExitFrameToken or VMFunction*.

Comment 1

[Jan de Mooij [:jandem]]

Attachment: Part 1 - Remove IonOOLPropertyOpExitFrameLayout and IonOOLSetterOpExitFrameLayout

Dead code since the Ion IC rewrite.

Comment 2

[Jan de Mooij [:jandem]]

Attachment: Part 2 - Refactor ExitFooterFrame

ExitFooterFrame now stores either the VMFunction* or the ExitFrameType.

I renamed ExitFrameToken to ExitFrameType because it's a better name now (ExitFrameType also contains VMFunction).

Comment 3

[Jan de Mooij [:jandem]]

Attachment: Part 2 - Refactor ExitFooterFrame

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c2e4e6d7886e
part 1 - Remove unused IonOOLPropertyOpExitFrameLayout and IonOOLSetterOpExitFrameLayout. r=nbp
https://hg.mozilla.org/integration/mozilla-inbound/rev/0b10cf1da7f2
part 2 - Shrink ExitFooterFrame to one word. r=nbp

Comment 5

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/c2e4e6d7886e
https://hg.mozilla.org/mozilla-central/rev/0b10cf1da7f2