libretro/ppsspp
1
0
Fork 0
mirror of https://github.com/libretro/ppsspp.git synced 2025-03-04 22:49:46 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

armjit: optimize reverse subtract, avoid temp imms.

Browse Source 
If we have a non-op2 imm, get rid of it asap.  If we have a op2 friendly
imm, keep it.
This commit is contained in:
Unknown W. Brackets 2013-11-08 18:31:49 -08:00
parent 415f22ecac
commit 945b8bf5c5
1 changed files with 22 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

40
Core/MIPS/ARM/ArmCompALU.cpp
View File

@ -193,32 +193,36 @@ namespace MIPSComp
{
if (gpr.IsImm(rs) && gpr.IsImm(rt)) {
gpr.SetImm(rd, (*eval)(gpr.GetImm(rs), gpr.GetImm(rt)));
} else if (gpr.IsImm(rt) || (gpr.IsImm(rs) && symmetric)) {
return;
}
if (gpr.IsImm(rt) || (gpr.IsImm(rs) && symmetric)) {
int lhs = gpr.IsImm(rs) ? rt : rs;
u32 rhsImm = gpr.IsImm(rs) ? gpr.GetImm(rs) : gpr.GetImm(rt);
gpr.MapDirtyIn(rd, lhs);
Operand2 op2;
// TODO: AND could be reversed, OR/EOR could use multiple ops (maybe still cheaper.)
if (TryMakeOperand2(rhsImm, op2)) {
gpr.MapDirtyIn(rd, lhs);
// MOV can avoid the ALU so might be faster?
if (useMOV && rhsImm == 0)
MOV(gpr.R(rd), gpr.R(lhs));
else
if (!useMOV || rhsImm != 0)
(this->*arith)(gpr.R(rd), gpr.R(lhs), op2);
} else {
// TODO: AND could be reversed, OR/EOR could use multiple ops.
MOVI2R(R0, rhsImm);
(this->*arith)(gpr.R(rd), gpr.R(lhs), R0);
else if (rd != lhs)
MOV(gpr.R(rd), gpr.R(lhs));
return;
}
} else if (gpr.IsImm(rs) && !symmetric) {
Operand2 op2;
// For SUB, we can use RSB as a reverse operation.
if (TryMakeOperand2(gpr.GetImm(rs), op2) && eval == &EvalSub) {
gpr.MapDirtyIn(rd, rt);
RSB(gpr.R(rd), gpr.R(rt), op2);
return;
}
} else if (gpr.IsImm(rs)) {
u32 rsImm = gpr.GetImm(rs);
gpr.MapDirtyIn(rd, rt);
MOVI2R(R0, rsImm);
(this->*arith)(gpr.R(rd), R0, gpr.R(rt));
} else {
// Generic solution
gpr.MapDirtyInIn(rd, rs, rt);
(this->*arith)(gpr.R(rd), gpr.R(rs), gpr.R(rt));
}
// Generic solution. If it's an imm, better to flush at this point.
gpr.MapDirtyInIn(rd, rs, rt);
(this->*arith)(gpr.R(rd), gpr.R(rs), gpr.R(rt));
}
void Jit::Comp_RType3(MIPSOpcode op)