Bug 27877

Summary: Allow custom memory allocation control for JavaScriptCore's PolymorphicAccessStructureList struct
Product: WebKit Reporter: Zoltan Horvath <zoltan>
Component: JavaScriptCoreAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, oliver
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: All   
Attachments:
Description Flags
proposed patch abarth: commit-queue+

Description Zoltan Horvath 2009-07-31 05:45:57 PDT
Inherits PolymorphicAccessStructureList struct from FastAllocBase because it has been instantiated by
'new' in JavaScriptCore/jit/JITStubs.cpp:1229.
Comment 1 Zoltan Horvath 2009-07-31 05:50:30 PDT
Created attachment 33874 [details]
proposed patch
Comment 2 Adam Barth 2009-08-01 21:40:07 PDT
Oliver: Here's the another patch similar to the one you didn't like.  Would you like to double check it?
Comment 3 Adam Barth 2009-08-05 23:27:32 PDT
Comment on attachment 33874 [details]
proposed patch

Olliej says this one is fine too.
Comment 4 Adam Barth 2009-08-06 12:17:54 PDT
Comment on attachment 33874 [details]
proposed patch

Clearing review flag on attachment: 33874

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	JavaScriptCore/ChangeLog
	M	JavaScriptCore/bytecode/Instruction.h
Committed r46854
	M	PlanetWebKit/config.ini
	M	PlanetWebKit/ChangeLog
r46853 = 7b9d89a3941dcdb8c85336726444abf5c7ac59f0 (trunk)
	M	JavaScriptCore/ChangeLog
	M	JavaScriptCore/bytecode/Instruction.h
r46854 = 018f29b6729ec241cc26692c306e2a4088dbbb85 (trunk)
First, rewinding head to replay your work on top of it...
Nothing to do.
http://trac.webkit.org/changeset/46854
Comment 5 Adam Barth 2009-08-06 12:17:58 PDT
All reviewed patches have been landed.  Closing bug.