Closed Bug 921140 Opened 7 years ago Closed 7 years ago

ZoneVector should default higher

Categories

(Core :: JavaScript Engine, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla27

People

(Reporter: terrence, Assigned: terrence)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Every runtime has at least an atom, self-hosting, and main zone. We should increase the inline storage for ZoneVector to remove at least one allocation from every runtime: e.g. to make worker startup faster on B2G.
Attachment #810712 - Flags: review?(wmccloskey)
Comment on attachment 810712 [details] [diff] [review]
zonevector_to_3_inline-v0.diff

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

OK. Also, I didn't realize we had a separate self-hosting zone. I don't think that's intentional (it wasn't on my part at least). We should try to use SystemZone for the self-hosting global to save more memory.
Attachment #810712 - Flags: review?(wmccloskey) → review+
Blocks: 916021
This bug is at loggerheads with bug 921213.
https://hg.mozilla.org/integration/mozilla-inbound/rev/1e6a27764acd

(In reply to Nicholas Nethercote [:njn] from comment #2)
> This bug is at loggerheads with bug 921213.

I don't think they need to be combined. I've landed the ZoneVector fix here and bug 921213 can work on the zone merging independently.
https://hg.mozilla.org/mozilla-central/rev/1e6a27764acd
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
You need to log in before you can comment on or make changes to this bug.