From 8e7b6f4178b80196a5d0047ee2ae33a211a36dd6 Mon Sep 17 00:00:00 2001 From: Sintendo Date: Sun, 5 Jan 2020 23:01:48 +0100 Subject: [PATCH 1/2] Jit64: addx - Prefer ADD over LEA when possible The old logic would always emit LEA when both sources are in a register and OE is disabled. However, ADD is still preferable when one of the sources matches the destination. Before: 45 8D 6C 35 00 lea r13d,[r13+rsi] After: 44 03 EE add r13d,esi --- Source/Core/Core/PowerPC/Jit64/Jit_Integer.cpp | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/Source/Core/Core/PowerPC/Jit64/Jit_Integer.cpp b/Source/Core/Core/PowerPC/Jit64/Jit_Integer.cpp index 8eadc3d33f..25b27e465b 100644 --- a/Source/Core/Core/PowerPC/Jit64/Jit_Integer.cpp +++ b/Source/Core/Core/PowerPC/Jit64/Jit_Integer.cpp @@ -1330,18 +1330,21 @@ void Jit64::addx(UGeckoInstruction inst) RCX64Reg Rd = gpr.Bind(d, RCMode::Write); RegCache::Realize(Ra, Rb, Rd); - if (Ra.IsSimpleReg() && Rb.IsSimpleReg() && !inst.OE) + if (d == a) { - LEA(32, Rd, MRegSum(Ra.GetSimpleReg(), Rb.GetSimpleReg())); + ADD(32, Rd, Rb); } else if (d == b) { ADD(32, Rd, Ra); } + else if (Ra.IsSimpleReg() && Rb.IsSimpleReg() && !inst.OE) + { + LEA(32, Rd, MRegSum(Ra.GetSimpleReg(), Rb.GetSimpleReg())); + } else { - if (d != a) - MOV(32, Rd, Ra); + MOV(32, Rd, Ra); ADD(32, Rd, Rb); } if (inst.OE) From 12fcbac2a32a9b97b72f919a7f25b76dda7532b1 Mon Sep 17 00:00:00 2001 From: Sintendo Date: Sun, 5 Jan 2020 23:28:04 +0100 Subject: [PATCH 2/2] Jit64: addx - Emit LEA for register + immediate Prefer LEA over MOV + ADD when dealing with immediates. Before: 44 8B EE mov r13d,esi 41 83 C5 20 add r13d,20h After: 44 8D 6E 20 lea r13d,[rsi+20h] --- Source/Core/Core/PowerPC/Jit64/Jit_Integer.cpp | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/Source/Core/Core/PowerPC/Jit64/Jit_Integer.cpp b/Source/Core/Core/PowerPC/Jit64/Jit_Integer.cpp index 25b27e465b..494969b71f 100644 --- a/Source/Core/Core/PowerPC/Jit64/Jit_Integer.cpp +++ b/Source/Core/Core/PowerPC/Jit64/Jit_Integer.cpp @@ -1342,6 +1342,14 @@ void Jit64::addx(UGeckoInstruction inst) { LEA(32, Rd, MRegSum(Ra.GetSimpleReg(), Rb.GetSimpleReg())); } + else if (Ra.IsSimpleReg() && Rb.IsImm() && !inst.OE) + { + LEA(32, Rd, MDisp(Ra.GetSimpleReg(), Rb.SImm32())); + } + else if (Rb.IsSimpleReg() && Ra.IsImm() && !inst.OE) + { + LEA(32, Rd, MDisp(Rb.GetSimpleReg(), Ra.SImm32())); + } else { MOV(32, Rd, Ra);