Bug 23026 - Move the deleted offsets vector into the PropertyMap
Summary: Move the deleted offsets vector into the PropertyMap
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-12-29 15:12 PST by Sam Weinig
Modified: 2008-12-29 16:29 PST (History)
0 users

See Also:


Attachments
patch (6.94 KB, patch)
2008-12-29 15:14 PST, Sam Weinig
oliver: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Sam Weinig 2008-12-29 15:12:04 PST
It is both rare to have deleted offsets and you are guaranteed to have a PropertMap if you have one, moving the Vector into the hashtable's struct would be a win for memory.
Comment 1 Sam Weinig 2008-12-29 15:14:33 PST
Created attachment 26304 [details]
patch
Comment 2 Oliver Hunt 2008-12-29 16:09:30 PST
Comment on attachment 26304 [details]
patch

r=me, assuming perf is good
Comment 3 Sam Weinig 2008-12-29 16:29:05 PST
Landed in r39502.