diff options
author | Fernando Sahmkow <fsahmkow27@gmail.com> | 2021-06-20 19:14:40 +0200 |
---|---|---|
committer | Fernando Sahmkow <fsahmkow27@gmail.com> | 2021-07-04 22:32:35 +0200 |
commit | a8a0927d424815d01782c6872a4c9ab605a87dbe (patch) | |
tree | 377585616778e1dd3fe423051442bd29bed02172 /src/video_core | |
parent | Texture Cache: Address feedback. (diff) | |
download | yuzu-a8a0927d424815d01782c6872a4c9ab605a87dbe.tar yuzu-a8a0927d424815d01782c6872a4c9ab605a87dbe.tar.gz yuzu-a8a0927d424815d01782c6872a4c9ab605a87dbe.tar.bz2 yuzu-a8a0927d424815d01782c6872a4c9ab605a87dbe.tar.lz yuzu-a8a0927d424815d01782c6872a4c9ab605a87dbe.tar.xz yuzu-a8a0927d424815d01782c6872a4c9ab605a87dbe.tar.zst yuzu-a8a0927d424815d01782c6872a4c9ab605a87dbe.zip |
Diffstat (limited to 'src/video_core')
-rw-r--r-- | src/video_core/texture_cache/image_base.h | 8 | ||||
-rw-r--r-- | src/video_core/texture_cache/texture_cache.h | 14 |
2 files changed, 11 insertions, 11 deletions
diff --git a/src/video_core/texture_cache/image_base.h b/src/video_core/texture_cache/image_base.h index 65e68cbb3..ff1feda9b 100644 --- a/src/video_core/texture_cache/image_base.h +++ b/src/video_core/texture_cache/image_base.h @@ -29,10 +29,10 @@ enum class ImageFlagBits : u32 { Sparse = 1 << 9, ///< Image has non continous submemory. // Garbage Collection Flags - BadOverlap = 1 << 10,///< This image overlaps other but doesn't fit, has higher - ///< garbage collection priority - Alias = 1 << 11, ///< This image has aliases and has priority on garbage - ///< collection + BadOverlap = 1 << 10, ///< This image overlaps other but doesn't fit, has higher + ///< garbage collection priority + Alias = 1 << 11, ///< This image has aliases and has priority on garbage + ///< collection }; DECLARE_ENUM_FLAG_OPERATORS(ImageFlagBits) diff --git a/src/video_core/texture_cache/texture_cache.h b/src/video_core/texture_cache/texture_cache.h index 500c4dd52..8b1aa8122 100644 --- a/src/video_core/texture_cache/texture_cache.h +++ b/src/video_core/texture_cache/texture_cache.h @@ -495,7 +495,7 @@ void TextureCache<P>::RunGarbageCollector() { } } if (True(image->flags & ImageFlagBits::Tracked)) { - UntrackImage(*image); + UntrackImage(*image, image_id); } UnregisterImage(image_id); DeleteImage(image_id); @@ -1474,16 +1474,16 @@ template <class P> template <typename Func> void TextureCache<P>::ForEachSparseSegment(ImageBase& image, Func&& func) { using FuncReturn = typename std::invoke_result<Func, GPUVAddr, VAddr, size_t>::type; - static constexpr bool BOOL_BREAK = std::is_same_v<FuncReturn, bool>; + static constexpr bool RETURNS_BOOL = std::is_same_v<FuncReturn, bool>; const auto segments = gpu_memory.GetSubmappedRange(image.gpu_addr, image.guest_size_bytes); for (auto& segment : segments) { const auto gpu_addr = segment.first; const auto size = segment.second; std::optional<VAddr> cpu_addr = gpu_memory.GpuToCpuAddress(gpu_addr); ASSERT(cpu_addr); - if constexpr (BOOL_BREAK) { + if constexpr (RETURNS_BOOL) { if (func(gpu_addr, *cpu_addr, size)) { - return true; + break; } } else { func(gpu_addr, *cpu_addr, size); @@ -1599,9 +1599,9 @@ void TextureCache<P>::UnregisterImage(ImageId image_id) { ASSERT(it != sparse_views.end()); auto& sparse_maps = it->second; for (auto& map_view_id : sparse_maps) { - const auto& map = slot_map_views[map_view_id]; - const VAddr cpu_addr = map.cpu_addr; - const std::size_t size = map.size; + const auto& map_range = slot_map_views[map_view_id]; + const VAddr cpu_addr = map_range.cpu_addr; + const std::size_t size = map_range.size; ForEachCPUPage(cpu_addr, size, [this, image_id](u64 page) { const auto page_it = page_table.find(page); if (page_it == page_table.end()) { |