Closed Bug 1643304 Opened 5 years ago Closed 5 years ago

Warp: fix try-catch support

Categories

(Core :: JavaScript Engine: JIT, task, P3)

task

Tracking

()

RESOLVED FIXED
mozilla79
Tracking Status
firefox79 --- fixed

People

(Reporter: jandem, Assigned: jandem)

References

Details

Attachments

(1 file)

Problem is that WarpBuilder skips the try-catch block but WarpOracle doesn't and aborts on the JSOp::Exception. For now the simplest fix is to add this to the list of supported ops and MOZ_CRASH in WarpBuilder.

WarpOracle was aborting on the JSOp::Exception. It's hard to (efficiently) skip
catch-blocks in WarpOracle too so this patch just adds it to the supported ops
and crashes in WarpBuilder since it's unreachable there.

This uncovered a few more test failures where we relied on specific exception messages.

Pushed by jdemooij@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0e128b8f949f Properly support try-catch in Warp. r=iain
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla79
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: