Closed Bug 1119694 Opened 6 years ago Closed 6 years ago

Remove the Allocator

Categories

(Core :: JavaScript: GC, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla38

People

(Reporter: lth, Assigned: lth)

References

Details

Attachments

(1 file)

The Allocator was a structure needed for PJS, with PJS gone it is no longer required, the arenas can be available directly from the context.

It will be easier to implement this change after we're done reorganizing the context hierarchy.
Depends on: 1118170
Attached patch Remove AllocatorSplinter Review
Sits on top of the context reorg in bug 1119292.
Attachment #8547521 - Flags: review?(terrence)
Comment on attachment 8547521 [details] [diff] [review]
Remove Allocator

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

Thanks!
Attachment #8547521 - Flags: review?(terrence) → review+
Try (after removing a spurious shadowing 'zone_' member): https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=756a70c50990
https://hg.mozilla.org/mozilla-central/rev/116f224b7789
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
You need to log in before you can comment on or make changes to this bug.