piwalker/dolphin
1
0
Fork 0
mirror of https://github.com/dolphin-emu/dolphin.git synced 2025-08-01 02:29:59 -06:00
Code Issues Packages Projects Releases Wiki Activity

Jit64: boolX - Precompute complement for eqvx

Browse Source 
In the case of eqvx, the final complement can always be baked directly
into the immediate value.

Before:
45 8B EF             mov         r13d,r15d
41 F7 D5             not         r13d
41 83 F5 04          xor         r13d,4

After:
45 8B EF             mov         r13d,r15d
41 83 F5 FB          xor         r13d,0FFFFFFFBh
This commit is contained in:
Sintendo
2021-01-28 23:06:00 +01:00
parent 26f70657bc
commit 3677a5035c
1 changed files with 2 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
Source/Core/Core/PowerPC/Jit64/Jit_Integer.cpp
View File

@ -668,8 +668,7 @@ void Jit64::boolX(UGeckoInstruction inst)
u32 imm = gpr.Imm32(i); u32 imm = gpr.Imm32(i);
bool complement_b = (inst.SUBOP10 == 60 /* andcx */) || (inst.SUBOP10 == 412 /* orcx */); bool complement_b = (inst.SUBOP10 == 60 /* andcx */) || (inst.SUBOP10 == 412 /* orcx */);
bool final_not = (inst.SUBOP10 == 476 /* nandx */) || (inst.SUBOP10 == 124 /* norx */) || bool final_not = (inst.SUBOP10 == 476 /* nandx */) || (inst.SUBOP10 == 124 /* norx */);
(inst.SUBOP10 == 284 /* eqvx */);
bool is_and = (inst.SUBOP10 == 28 /* andx */) || (inst.SUBOP10 == 60 /* andcx */) || bool is_and = (inst.SUBOP10 == 28 /* andx */) || (inst.SUBOP10 == 60 /* andcx */) ||
(inst.SUBOP10 == 476 /* nandx */); (inst.SUBOP10 == 476 /* nandx */);
bool is_or = (inst.SUBOP10 == 444 /* orx */) || (inst.SUBOP10 == 412 /* orcx */) || bool is_or = (inst.SUBOP10 == 444 /* orx */) || (inst.SUBOP10 == 412 /* orcx */) ||
@ -677,7 +676,7 @@ void Jit64::boolX(UGeckoInstruction inst)
bool is_xor = (inst.SUBOP10 == 316 /* xorx */) || (inst.SUBOP10 == 284 /* eqvx */); bool is_xor = (inst.SUBOP10 == 316 /* xorx */) || (inst.SUBOP10 == 284 /* eqvx */);
// Precompute complement when possible // Precompute complement when possible
if (complement_b && gpr.IsImm(b)) if (complement_b && gpr.IsImm(b) || (inst.SUBOP10 == 284 /* eqvx */))
{ {
imm = ~imm; imm = ~imm;
complement_b = false; complement_b = false;
@ -692,12 +691,6 @@ void Jit64::boolX(UGeckoInstruction inst)
if (a != j) if (a != j)
MOV(32, Ra, Rj); MOV(32, Ra, Rj);
XOR(32, Ra, Imm32(imm)); XOR(32, Ra, Imm32(imm));
if (final_not)
{
NOT(32, Ra);
needs_test = true;
}
} }
else if (is_and) else if (is_and)
{ {