Simplify JSOP_LOOPENTRY handling in IonBuilder

RESOLVED FIXED in Firefox 62

Status

()

enhancement
RESOLVED FIXED
Last year
6 months ago

People

(Reporter: jandem, Assigned: jandem)

Tracking

unspecified
mozilla62
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox62 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

Last year
Posted patch PatchSplinter Review
The CFG generator skips the JSOP_LOOPENTRY ops, and IonBuilder has to call initLoopEntry to insert interrupt checks. It's much simpler to not skip these ops in the first place and then JSOP_LOOPENTRY can be compiled by IonBuilder like any other op and this way we always know its pc.

I ran into this while working on bug 1382650 and it's also causing problems for bhackett in bug 1465289, so I split off my fix for this from the patch for bug 1382650.
Attachment #8984188 - Flags: review?(bhackett1024)
Assignee

Comment 1

Last year
I tested this on some benchmarks and verified for/while/do-while loops still enter Ion via OSR and we have a single interrupt check.
Comment on attachment 8984188 [details] [diff] [review]
Patch

Review of attachment 8984188 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks!
Attachment #8984188 - Flags: review?(bhackett1024) → review+

Comment 3

Last year
Pushed by jandemooij@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c9ee59766684
Simplify JSOP_LOOPENTRY handling in IonBuilder. r=bhackett

Comment 4

Last year
bugherder
https://hg.mozilla.org/mozilla-central/rev/c9ee59766684
Status: ASSIGNED → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
Depends on: 1514625
You need to log in before you can comment on or make changes to this bug.