885112 - IonMonkey: MFunctionEnvironment should be movable and alias nothing

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Kannan Vijayan [:djvj]]

A function's environment object should never change.  MFunctionEnvironment should be movable and subject to LICM.

Comment 1

[Kannan Vijayan [:djvj]]

Attachment: Patch.

This improves the performance a compiled forEach with an inlined, scope-using callback, by 7%.

Comment 2

[Kannan Vijayan [:djvj]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/3d389787101e

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/3d389787101e