Bug 22272

Summary: The JIT code for calls is HUUUUUUUGE!
Product: WebKit Reporter: Gavin Barraclough <barraclough>
Component: JavaScriptCoreAssignee: Gavin Barraclough <barraclough>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.5   
Attachments:
Description Flags
The patch ggaren: review+

Gavin Barraclough
Reported 2008-11-14 15:59:05 PST
We should generate less of it.
Attachments
The patch (20.62 KB, patch)
2008-11-14 16:03 PST, Gavin Barraclough 		ggaren: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Gavin Barraclough
Comment 1 2008-11-14 16:03:48 PST
Created attachment 25178 [details] The patch 1% progression on v8 tests
Geoffrey Garen
Comment 2 2008-11-14 16:21:41 PST
Comment on attachment 25178 [details] The patch emitRetrieveArg This is a fishy name. The problem is really that emitGetArg is a bad name. So, we should rename emitGetArg to emitGetVirtualRegister. Probably a good thing to do after landing this patch. r=me
Gavin Barraclough
Comment 3 2008-11-14 17:37:23 PST
Sending JavaScriptCore/ChangeLog Sending JavaScriptCore/VM/CTI.cpp Sending JavaScriptCore/VM/CTI.h Sending JavaScriptCore/VM/CodeBlock.h Sending JavaScriptCore/VM/Machine.cpp Sending JavaScriptCore/VM/Machine.h Transmitting file data ...... Committed revision 38412.
Note You need to log in before you can comment on or make changes to this bug.