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

PowerPC: Update variable name conventions and const-ness around calls to JitCache_TranslateAddress().

This commit is contained in:
Admiral H. Curtiss 2021-08-09 01:18:29 +02:00
parent 3296d2fc1f
commit 95fbd09691
1 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
Source/Core/Core/PowerPC/JitCommon/JitCache.cpp
View File

@ -96,10 +96,10 @@ void JitBaseBlockCache::RunOnBlocks(std::function<void(const JitBlock&)> f)
JitBlock* JitBaseBlockCache::AllocateBlock(u32 em_address) JitBlock* JitBaseBlockCache::AllocateBlock(u32 em_address)
{ {
u32 physicalAddress = PowerPC::JitCache_TranslateAddress(em_address).address; const u32 physical_address = PowerPC::JitCache_TranslateAddress(em_address).address;
JitBlock& b = block_map.emplace(physicalAddress, JitBlock())->second; JitBlock& b = block_map.emplace(physical_address, JitBlock())->second;
b.effectiveAddress = em_address; b.effectiveAddress = em_address;
b.physicalAddress = physicalAddress; b.physicalAddress = physical_address;
b.msrBits = MSR.Hex & JIT_CACHE_MSR_MASK; b.msrBits = MSR.Hex & JIT_CACHE_MSR_MASK;
b.linkData.clear(); b.linkData.clear();
b.fast_block_map_index = 0; b.fast_block_map_index = 0;
@ -185,25 +185,25 @@ const u8* JitBaseBlockCache::Dispatch()
void JitBaseBlockCache::InvalidateICache(u32 address, u32 length, bool forced) void JitBaseBlockCache::InvalidateICache(u32 address, u32 length, bool forced)
{ {
auto translated = PowerPC::JitCache_TranslateAddress(address); const auto translated = PowerPC::JitCache_TranslateAddress(address);
if (!translated.valid) if (!translated.valid)
return; return;
u32 pAddr = translated.address; const u32 physical_address = translated.address;
// Optimize the common case of length == 32 which is used by Interpreter::dcb* // Optimize the common case of length == 32 which is used by Interpreter::dcb*
bool destroy_block = true; bool destroy_block = true;
if (length == 32) if (length == 32)
{ {
if (!valid_block.Test(pAddr / 32)) if (!valid_block.Test(physical_address / 32))
destroy_block = false; destroy_block = false;
else else
valid_block.Clear(pAddr / 32); valid_block.Clear(physical_address / 32);
} }
if (destroy_block) if (destroy_block)
{ {
// destroy JIT blocks // destroy JIT blocks
ErasePhysicalRange(pAddr, length); ErasePhysicalRange(physical_address, length);
// If the code was actually modified, we need to clear the relevant entries from the // If the code was actually modified, we need to clear the relevant entries from the
// FIFO write address cache, so we don't end up with FIFO checks in places they shouldn't // FIFO write address cache, so we don't end up with FIFO checks in places they shouldn't