Bug 20900

Summary: More JIT code generation for op_call
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+

Description Gavin Barraclough 2008-09-17 11:13:59 PDT 
Optimizations for op_call in CTI.  Move check for (ctiCode == 0) into JIT code, move copying of scopeChain for CodeBlocks that needFullScopeChain into head of functions, instead of checking prior to making the call.

3% on v8-tests (4% on richards, 6% in delta-blue)
Comment 1 Gavin Barraclough 2008-09-17 11:48:29 PDT 
Created attachment 23505 [details]
The patch
Comment 2 Geoffrey Garen 2008-09-17 12:22:02 PDT 
Comment on attachment 23505 [details]
The patch

r=me, with some comments on irc.
Comment 3 Gavin Barraclough 2008-09-17 12:30:53 PDT 
Sending        JavaScriptCore/ChangeLog
Sending        JavaScriptCore/VM/CTI.cpp
Sending        JavaScriptCore/VM/Machine.cpp
Sending        JavaScriptCore/VM/Machine.h
Transmitting file data ....
Committed revision 36544.