Closed
Bug 811176
Opened 12 years ago
Closed 12 years ago
Clean up JIT code more aggressively
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
People
(Reporter: billm, Assigned: billm)
References
Details
(Whiteboard: [MemShrink:P2][soft-blocker])
Attachments
(2 files)
2.01 KB,
patch
|
terrence
:
review+
|
Details | Diff | Splinter Review |
3.08 KB,
patch
|
n.nethercote
:
review+
|
Details | Diff | Splinter Review |
We have a special GC mode for "clean up everything". However, it's not getting triggered for shrinking GCs. This patch fixes that.
Attachment #680911 -
Flags: review?(terrence)
Assignee | ||
Comment 1•12 years ago
|
||
Right now we hang on to small executable pools forever. This drops our references to them after every GC.
Attachment #680912 -
Flags: review?(n.nethercote)
![]() |
||
Updated•12 years ago
|
Attachment #680912 -
Flags: review?(n.nethercote) → review+
Comment 2•12 years ago
|
||
Adding a direct slim-fast dependency so this is easier to find. Thanks a lot for looking into this, Bill!
Blocks: slim-fast
Comment 3•12 years ago
|
||
Comment on attachment 680911 [details] [diff] [review]
clean up on shrinking GCs
Review of attachment 680911 [details] [diff] [review]:
-----------------------------------------------------------------
Good find!
Attachment #680911 -
Flags: review?(terrence) → review+
![]() |
||
Updated•12 years ago
|
Whiteboard: [MemShrink] → [MemShrink:P2]
Assignee | ||
Comment 4•12 years ago
|
||
Comment 5•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/6e56236dfd3f
https://hg.mozilla.org/mozilla-central/rev/5a1edbdb251f
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
Updated•12 years ago
|
blocking-basecamp: --- → +
Whiteboard: [MemShrink:P2] → [MemShrink:P2][soft-blocker]
Assignee | ||
Comment 6•12 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/cdf8d346dab0
https://hg.mozilla.org/releases/mozilla-aurora/rev/3048a302c699
status-firefox18:
--- → fixed
status-firefox19:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•