Bug 22246

Summary: Get arguments for opcodes together to eliminate more redundant memory reads
Product: WebKit Reporter: Cameron Zwarich (cpst) <zwarich>
Component: JavaScriptCoreAssignee: Cameron Zwarich (cpst) <zwarich>
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Description Flags
Proposed patch darin: review+

Description Cameron Zwarich (cpst) 2008-11-13 16:20:49 PST
Currently, if an opcode gets its first operand in eax and its second operand in edx, the read from eax will kill the last result register. Instead, if there is an opportunity for optimization for the second operand we should move eax to edx and then read the first operand from memory into eax.
Comment 1 Cameron Zwarich (cpst) 2008-11-13 16:29:59 PST
Created attachment 25142 [details]
Proposed patch
Comment 2 Darin Adler 2008-11-13 16:33:39 PST
Comment on attachment 25142 [details]
Proposed patch

Comment 3 Cameron Zwarich (cpst) 2008-11-13 17:40:30 PST
Landed in r38379.