Closed Bug 1543592 Opened 3 years ago Closed 3 years ago

Some JSOP_DEBUGAFTERYIELD changes

Categories

(Core :: JavaScript Engine, task, P2)

task

Tracking

()

RESOLVED FIXED
mozilla68
Tracking Status
firefox68 --- fixed

People

(Reporter: jandem, Assigned: jandem)

References

Details

Attachments

(2 files)

I want to rename it to JSOP_AFTERYIELD (the debugger-only ship has sailed with AbstractGeneratorObject::isAfterYieldOrAwait) and make it a jump target op so that the interpreter will correctly restore its interpreterICEntry field when resuming a generator.

This will help the Baseline interpreter restore its interpreterICEntry field
without calling into C++.

Depends on D27032

Priority: -- → P2
Pushed by jdemooij@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4584d95fcaea
part 1 - Rename JSOP_DEBUGAFTERYIELD to JSOP_AFTERYIELD. r=tcampbell
https://hg.mozilla.org/integration/autoland/rev/c35e1a0a6cfb
part 2 - Make JSOP_AFTERYIELD a jump target op. r=tcampbell
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
You need to log in before you can comment on or make changes to this bug.