IonMonkey clean-up: rewrite generateBailoutTail in C++

RESOLVED INVALID

Status

()

Core
JavaScript Engine
RESOLVED INVALID
5 years ago
4 years ago

People

(Reporter: nbp, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

5 years ago
generateBailoutTail is doing some kind of switch case with many callWithABI and use the returned value to know if we should fallback into handleException.  We should rewrite this switch case in C++ and only keep one callWithABI and the handleException at the end.

The extracted C++ function will handle the post-bailout processing with the error code returned by the bailout.
(Reporter)

Comment 1

4 years ago
Apparently the code changed a lot since the landing of Baseline and now it is shared between all platforms and perform a masm.enterFakeExitFrame, which should better remain in the macro assembler.
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.