Closed Bug 903754 Opened 7 years ago Closed 7 years ago

Remove InterpreterFrames class and use InterpreterActivation instead

Categories

(Core :: JavaScript Engine, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla26

People

(Reporter: jandem, Assigned: jandem)

Details

Attachments

(1 file, 1 obsolete file)

Attached patch Patch (obsolete) — Splinter Review
InterpreterFrames is used to enable interrupts in js::Interpret invocations. The linked list of (interpreter) activations we now have has everything we need though, so there's no need to maintain a second list.
Attachment #788511 - Flags: review?(luke)
Forgot to mention, the patch also removes InterpreterActivation::current_, we can use regs_.fp() instead.
Nice find; I forgot all about this!  Maybe I'm missing a constraint, but could InterpreterActivation just store the switchMask/value directly such that enableInterrupts could *switchMask = value directly?
Attached patch Patch v2Splinter Review
(In reply to Luke Wagner [:luke] from comment #2)
> Nice find; I forgot all about this!  Maybe I'm missing a constraint, but
> could InterpreterActivation just store the switchMask/value directly such
> that enableInterrupts could *switchMask = value directly?

Yeah that works. This patch removes the InterruptEnabler stuff as well.
Attachment #788511 - Attachment is obsolete: true
Attachment #788511 - Flags: review?(luke)
Attachment #789011 - Flags: review?(luke)
Comment on attachment 789011 [details] [diff] [review]
Patch v2

Great
Attachment #789011 - Flags: review?(luke) → review+
https://hg.mozilla.org/mozilla-central/rev/cd086a74f522
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
You need to log in before you can comment on or make changes to this bug.