Closed Bug 1055758 Opened 10 years ago Closed 10 years ago

rm CompilerRoot

Categories

(Core :: JavaScript Engine: JIT, defect)

Product:
Core
Component:
JavaScript Engine: JIT
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla34

People

(Reporter: jandem, Assigned: jandem)

Details

Attachments

(1 file)

Patch
30.55 KB, patch
bhackett1024
: review+
Details | Diff | Splinter Review
Attached patch Patch — — Splinter Review 
We can no longer GC during Ion compilation, so maintaining this list of roots is unnecessary. We do want to keep the non-nursery-thing assert, so this patch replaces CompilerRoot with an AlwaysTenured<T> class.

 9 files changed, 76 insertions(+), 175 deletions(-)
Attachment #8475391 - Flags: review?(bhackett1024)
Attachment #8475391 - Flags: review?(bhackett1024) → review+
https://hg.mozilla.org/mozilla-central/rev/970afb572151
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: