dburr/eden
1
0
Fork 0
mirror of https://git.eden-emu.dev/eden-emu/eden.git synced 2025-07-24 11:25:46 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

buffer_cache: Eliminate redundant map lookup in MarkRegionAsWritten()

Browse source 
We can make use of emplace()'s return value to determine whether or not
we need to perform an increment.

emplace() performs no insertion if an element already exist, so this can
eliminate a find() call.
This commit is contained in:
Lioncash 2020-07-20 17:40:04 -04:00
parent 5714c2c44a
commit 1842c5df37
1 changed files with 3 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

9
src/video_core/buffer_cache/buffer_cache.h
View file

@ -524,11 +524,8 @@ private:
void MarkRegionAsWritten(VAddr start, VAddr end) {
const u64 page_end = end >> WRITE_PAGE_BIT;
for (u64 page_start = start >> WRITE_PAGE_BIT; page_start <= page_end; ++page_start) {
auto it = written_pages.find(page_start);
if (it != written_pages.end()) {
it->second = it->second + 1;
} else {
written_pages.insert_or_assign(page_start, 1);
if (const auto [it, inserted] = written_pages.emplace(page_start, 1); !inserted) {
++it->second;
}
}
}
@ -539,7 +536,7 @@ private:
auto it = written_pages.find(page_start);
if (it != written_pages.end()) {
if (it->second > 1) {
it->second = it->second - 1;
--it->second;
} else {
written_pages.erase(it);
}