Bug 151012

Summary: MacroAssemblerX86_64 should expose the full power of LEA
Product: WebKit Reporter: Filip Pizlo <fpizlo>
Component: JavaScriptCoreAssignee: Nobody <webkit-unassigned>
Status: NEW    
Severity: Normal CC: benjamin, kling
Priority: P2    
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 154319    

Filip Pizlo
Reported 2015-11-08 18:25:17 PST
It should be possible to do things like: m_jit.leaPtr(MacroAssembler::BaseIndex(rax, rcx), rdx); // three-operand form of addPtr m_jit.lea32(MacroAssembler::BaseIndex(rax, rcx), rdx); // three-operand form of add32 m_jit.add32(rax, rcx, rdx); // this uses lea32, or vice-versa. it's useful to have this alias m_jit.leaPtr(MacroAssembler::BaseIndex(rax, rcx, ScaleEight, 42), rdx); // MEGA COMBO
Attachments
Add attachment proposed patch, testcase, etc.
Benjamin Poulain
Comment 1 2015-11-09 16:24:09 PST
Andreas, can you help?
Note You need to log in before you can comment on or make changes to this bug.