Closed Bug 837073 Opened 11 years ago Closed 11 years ago

IonMonkey clean-up: rewrite generateBailoutTail in C++

Categories

(Core :: JavaScript Engine, defect)

Product:
Core
Component:
JavaScript Engine
Platform:
x86_64
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED INVALID

People

(Reporter: nbp, Unassigned)

Details

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.
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
Closed: 11 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.