Summary: | EdenCollections unnecessarily visit SmallStrings | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Mark Hahnenberg <mhahnenb> | ||||||||
Component: | JavaScriptCore | Assignee: | Mark Hahnenberg <mhahnenb> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | commit-queue, kling | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 140774 | ||||||||||
Attachments: |
|
Description
Mark Hahnenberg
2015-01-21 23:17:32 PST
Created attachment 245125 [details]
Patch
Created attachment 245126 [details]
GC logs before
Created attachment 245127 [details]
GC logs after
Attached GC timing logs before and after the change. Time spent in VisitSmallStrings is down significantly, but it didn't seem to affect the overall Eden time as much as I'd expect. Comment on attachment 245125 [details]
Patch
r=me
Comment on attachment 245125 [details] Patch Clearing flags on attachment: 245125 Committed r178984: <http://trac.webkit.org/changeset/178984> All reviewed patches have been landed. Closing bug. (In reply to comment #4) > Attached GC timing logs before and after the change. Time spent in > VisitSmallStrings is down significantly, but it didn't seem to affect the > overall Eden time as much as I'd expect. Still, that's a really impressive improvement! Thanks for making this change, Mark. :) |