From 4b95b0df973027f4682549b0bd27bf9e05d8155f Mon Sep 17 00:00:00 2001 From: bunnei Date: Thu, 27 May 2021 14:47:24 -0700 Subject: video_core: rasterizer_cache: Use u16 for cached page count. - Greatly reduces the risk of overflow, at the cost of doubling the size of this array. --- src/video_core/rasterizer_accelerated.h | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) (limited to 'src/video_core/rasterizer_accelerated.h') diff --git a/src/video_core/rasterizer_accelerated.h b/src/video_core/rasterizer_accelerated.h index 9227a4adc..ea879bfdd 100644 --- a/src/video_core/rasterizer_accelerated.h +++ b/src/video_core/rasterizer_accelerated.h @@ -29,20 +29,20 @@ private: public: CacheEntry() = default; - std::atomic_uint8_t& Count(std::size_t page) { - return values[page & 7]; + std::atomic_uint16_t& Count(std::size_t page) { + return values[page & 3]; } - const std::atomic_uint8_t& Count(std::size_t page) const { - return values[page & 7]; + const std::atomic_uint16_t& Count(std::size_t page) const { + return values[page & 3]; } private: - std::array values{}; + std::array values{}; }; static_assert(sizeof(CacheEntry) == 8, "CacheEntry should be 8 bytes!"); - std::array cached_pages; + std::array cached_pages; Core::Memory::Memory& cpu_memory; }; -- cgit v1.2.3