Bug 22748

Summary: Port the op_j?n?eq_null JIT code generation to use the MacroAssembler.
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 oliver: review+

Gavin Barraclough
Reported 2008-12-08 20:22:55 PST
Clean up slightly at the same time. The 'j' forms currently compare, then set a register, then compare again, then branch. Branch directly on the result of the first compare.
Attachments
The patch (8.10 KB, patch)
2008-12-08 20:30 PST, Gavin Barraclough 		oliver: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Gavin Barraclough
Comment 1 2008-12-08 20:30:44 PST
Created attachment 25869 [details] The patch Around a 1% progression on deltablue, crypto & early boyer, for about 1/2% overall on v8-tests.
Gavin Barraclough
Comment 2 2008-12-08 20:54:23 PST
Sending JavaScriptCore/ChangeLog Sending JavaScriptCore/jit/JIT.cpp Sending JavaScriptCore/jit/JITPropertyAccess.cpp Transmitting file data ... Committed revision 39122.
Note You need to log in before you can comment on or make changes to this bug.