diff --git a/Source/Core/VideoBackends/Software/EfbInterface.cpp b/Source/Core/VideoBackends/Software/EfbInterface.cpp index a13feb6c50..bb3cfeda90 100644 --- a/Source/Core/VideoBackends/Software/EfbInterface.cpp +++ b/Source/Core/VideoBackends/Software/EfbInterface.cpp @@ -5,6 +5,7 @@ #include "VideoBackends/Software/EfbInterface.h" #include +#include #include #include #include @@ -18,11 +19,11 @@ #include "VideoCommon/LookUpTables.h" #include "VideoCommon/PerfQueryBase.h" -static u8 efb[EFB_WIDTH * EFB_HEIGHT * 6]; - namespace EfbInterface { -u32 perf_values[PQ_NUM_MEMBERS]; +static std::array efb; + +static std::array perf_values; static inline u32 GetColorOffset(u16 x, u16 y) { @@ -31,7 +32,9 @@ static inline u32 GetColorOffset(u16 x, u16 y) static inline u32 GetDepthOffset(u16 x, u16 y) { - return (x + y * EFB_WIDTH) * 3 + DEPTH_BUFFER_START; + constexpr u32 depth_buffer_start = EFB_WIDTH * EFB_HEIGHT * 3; + + return (x + y * EFB_WIDTH) * 3 + depth_buffer_start; } static void SetPixelAlphaOnly(u32 offset, u8 a) @@ -682,4 +685,27 @@ bool ZCompare(u16 x, u16 y, u32 z) return pass; } + +u32 GetPerfQueryResult(PerfQueryType type) +{ + return perf_values[type]; +} + +void ResetPerfQuery() +{ + perf_values = {}; +} + +void IncPerfCounterQuadCount(PerfQueryType type) +{ + // NOTE: hardware doesn't process individual pixels but quads instead. + // Current software renderer architecture works on pixels though, so + // we have this "quad" hack here to only increment the registers on + // every fourth rendered pixel + static u32 quad[PQ_NUM_MEMBERS]; + if (++quad[type] != 3) + return; + quad[type] = 0; + ++perf_values[type]; +} } diff --git a/Source/Core/VideoBackends/Software/EfbInterface.h b/Source/Core/VideoBackends/Software/EfbInterface.h index 7f7c0ec608..518395d7c9 100644 --- a/Source/Core/VideoBackends/Software/EfbInterface.h +++ b/Source/Core/VideoBackends/Software/EfbInterface.h @@ -4,16 +4,12 @@ #pragma once -#include - #include "Common/CommonTypes.h" #include "VideoCommon/PerfQueryBase.h" #include "VideoCommon/VideoCommon.h" namespace EfbInterface { -const int DEPTH_BUFFER_START = EFB_WIDTH * EFB_HEIGHT * 3; - // xfb color format - packed so the compiler doesn't mess with alignment #pragma pack(push, 1) struct yuv422_packed @@ -61,17 +57,7 @@ u8* GetPixelPointer(u16 x, u16 y, bool depth); void EncodeXFB(u8* xfb_in_ram, u32 memory_stride, const EFBRectangle& source_rect, float y_scale, float gamma); -extern u32 perf_values[PQ_NUM_MEMBERS]; -inline void IncPerfCounterQuadCount(PerfQueryType type) -{ - // NOTE: hardware doesn't process individual pixels but quads instead. - // Current software renderer architecture works on pixels though, so - // we have this "quad" hack here to only increment the registers on - // every fourth rendered pixel - static u32 quad[PQ_NUM_MEMBERS]; - if (++quad[type] != 3) - return; - quad[type] = 0; - ++perf_values[type]; -} -} +u32 GetPerfQueryResult(PerfQueryType type); +void ResetPerfQuery(); +void IncPerfCounterQuadCount(PerfQueryType type); +} // namespace EfbInterface diff --git a/Source/Core/VideoBackends/Software/SWmain.cpp b/Source/Core/VideoBackends/Software/SWmain.cpp index 8765a28e4c..18a739726b 100644 --- a/Source/Core/VideoBackends/Software/SWmain.cpp +++ b/Source/Core/VideoBackends/Software/SWmain.cpp @@ -38,11 +38,8 @@ public: ~PerfQuery() {} void EnableQuery(PerfQueryGroup type) override {} void DisableQuery(PerfQueryGroup type) override {} - void ResetQuery() override - { - memset(EfbInterface::perf_values, 0, sizeof(EfbInterface::perf_values)); - } - u32 GetQueryResult(PerfQueryType type) override { return EfbInterface::perf_values[type]; } + void ResetQuery() override { EfbInterface::ResetPerfQuery(); } + u32 GetQueryResult(PerfQueryType type) override { return EfbInterface::GetPerfQueryResult(type); } void FlushResults() override {} bool IsFlushed() const override { return true; } };