ShuriZma
/
dolphin
mirror of https://github.com/dolphin-emu/dolphin.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #12377 from JosJuice/jitarm64-a-d-discard-guard 

JitArm64: Guard against discarding a in lmw/stmw
This commit is contained in:
Mai 2023-12-10 12:19:41 -05:00 committed by GitHub
parent d272b0ef84 13a4f1b799
commit db1620db42
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 28 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
Source/Core/Core/PowerPC/JitArm64/JitArm64_LoadStore.cpp
View File

@ -541,6 +541,19 @@ void JitArm64::lmw(UGeckoInstruction inst)
if (!a_is_addr_base_reg) if (!a_is_addr_base_reg)
MOV(addr_base_reg, addr_reg); MOV(addr_base_reg, addr_reg);
BitSet32 gprs_to_discard{};
if (!jo.memcheck)
{
gprs_to_discard = js.op->gprDiscardable;
if (gprs_to_discard[a])
{
if (a_is_addr_base_reg)
gprs_to_discard[a] = false;
else
gpr.DiscardRegisters(BitSet32{int(a)});
}
}
BitSet32 gprs_to_flush = ~js.op->gprInUse & BitSet32(0xFFFFFFFFU << d); BitSet32 gprs_to_flush = ~js.op->gprInUse & BitSet32(0xFFFFFFFFU << d);
if (!js.op->gprInUse[a]) if (!js.op->gprInUse[a])
{ {
@ -592,7 +605,7 @@ void JitArm64::lmw(UGeckoInstruction inst)
// after this instruction, flush registers that would be flushed after this instruction anyway. // after this instruction, flush registers that would be flushed after this instruction anyway.
// //
// We try to store two registers at a time when possible to let the register cache use STP. // We try to store two registers at a time when possible to let the register cache use STP.
if (!jo.memcheck && js.op->gprDiscardable[i]) if (gprs_to_discard[i])
{ {
gpr.DiscardRegisters(BitSet32{int(i)}); gpr.DiscardRegisters(BitSet32{int(i)});
} }
@ -645,6 +658,19 @@ void JitArm64::stmw(UGeckoInstruction inst)
if (!a_is_addr_base_reg) if (!a_is_addr_base_reg)
MOV(addr_base_reg, addr_reg); MOV(addr_base_reg, addr_reg);
BitSet32 gprs_to_discard{};
if (!jo.memcheck)
{
gprs_to_discard = js.op->gprDiscardable;
if (gprs_to_discard[a])
{
if (a_is_addr_base_reg)
gprs_to_discard[a] = false;
else
gpr.DiscardRegisters(BitSet32{int(a)});
}
}
const BitSet32 dirty_gprs_to_flush_unmasked = ~js.op->gprInUse & gpr.GetDirtyGPRs(); const BitSet32 dirty_gprs_to_flush_unmasked = ~js.op->gprInUse & gpr.GetDirtyGPRs();
BitSet32 dirty_gprs_to_flush = dirty_gprs_to_flush_unmasked & BitSet32(0xFFFFFFFFU << s); BitSet32 dirty_gprs_to_flush = dirty_gprs_to_flush_unmasked & BitSet32(0xFFFFFFFFU << s);
if (dirty_gprs_to_flush_unmasked[a]) if (dirty_gprs_to_flush_unmasked[a])
@ -692,7 +718,7 @@ void JitArm64::stmw(UGeckoInstruction inst)
// after this instruction, flush registers that would be flushed after this instruction anyway. // after this instruction, flush registers that would be flushed after this instruction anyway.
// //
// We try to store two registers at a time when possible to let the register cache use STP. // We try to store two registers at a time when possible to let the register cache use STP.
if (!jo.memcheck && js.op->gprDiscardable[i]) if (gprs_to_discard[i])
{ {
gpr.DiscardRegisters(BitSet32{int(i)}); gpr.DiscardRegisters(BitSet32{int(i)});
} }