838520 - BaselineCompiler: Exception handling and forced returns have to unwind scopes

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Jan de Mooij [:jandem]]

Will fix jit-tests/tests/auto-regress/bug637010.js

Comment 1

[Jan de Mooij [:jandem]]

Attachment: Part 1: Refactor UnwindScope to use AbstractFramePtr

The m-c part.

Comment 2

[Jan de Mooij [:jandem]]

Attachment: Part 1: Refactor UnwindScope to use AbstractFramePtr

Oops, forgot to qref.

Comment 3

[Jan de Mooij [:jandem]]

Attachment: Part 2: Baseline compiler part

Initializing evalPrev is a bit unfortunate, but doing it in ScopeIter can lead to O(n) behavior and there are only two places where we have to do this, so I think it's the best solution.

Comment 4

[Jan de Mooij [:jandem]]

Part 1:

https://hg.mozilla.org/integration/mozilla-inbound/rev/cb97c0155552

Comment 5

[Jan de Mooij [:jandem]]

Part 2:

https://hg.mozilla.org/projects/ionmonkey/rev/06ed972b5d4d

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/cb97c0155552