From 5635d4b709a103fd8fcef2ed9be615296cb020e7 Mon Sep 17 00:00:00 2001 From: Lioncash Date: Thu, 18 Aug 2016 22:35:58 -0400 Subject: [PATCH] Fifo: Make SyncGPUReason an enum class --- Source/Core/VideoCommon/CommandProcessor.cpp | 2 +- Source/Core/VideoCommon/Fifo.cpp | 6 +++--- Source/Core/VideoCommon/Fifo.h | 16 ++++++++-------- Source/Core/VideoCommon/MainBase.cpp | 6 +++--- 4 files changed, 15 insertions(+), 15 deletions(-) diff --git a/Source/Core/VideoCommon/CommandProcessor.cpp b/Source/Core/VideoCommon/CommandProcessor.cpp index 0cf8dd3b7b..287d93d28f 100644 --- a/Source/Core/VideoCommon/CommandProcessor.cpp +++ b/Source/Core/VideoCommon/CommandProcessor.cpp @@ -238,7 +238,7 @@ void RegisterMMIO(MMIO::Mapping* mmio, u32 base) MMIO::DirectRead(MMIO::Utils::HighPart(&fifo.CPReadWriteDistance)), MMIO::ComplexWrite([](u32, u16 val) { WriteHigh(fifo.CPReadWriteDistance, val); - Fifo::SyncGPU(Fifo::SYNC_GPU_OTHER); + Fifo::SyncGPU(Fifo::SyncGPUReason::Other); if (fifo.CPReadWriteDistance == 0) { GPFifo::ResetGatherPipe(); diff --git a/Source/Core/VideoCommon/Fifo.cpp b/Source/Core/VideoCommon/Fifo.cpp index 8d3df111e4..cfb0d310a2 100644 --- a/Source/Core/VideoCommon/Fifo.cpp +++ b/Source/Core/VideoCommon/Fifo.cpp @@ -92,7 +92,7 @@ void PauseAndLock(bool doLock, bool unpauseOnUnlock) { if (doLock) { - SyncGPU(SYNC_GPU_OTHER); + SyncGPU(SyncGPUReason::Other); EmulatorState(false); FlushGpu(); } @@ -199,7 +199,7 @@ void PushFifoAuxBuffer(void* ptr, size_t size) { if (size > (size_t)(s_fifo_aux_data + FIFO_SIZE - s_fifo_aux_write_ptr)) { - SyncGPU(SYNC_GPU_AUX_SPACE, /* may_move_read_ptr */ false); + SyncGPU(SyncGPUReason::AuxSpace, /* may_move_read_ptr */ false); if (!s_gpu_mainloop.IsRunning()) { // GPU is shutting down @@ -255,7 +255,7 @@ static void ReadDataFromFifoOnCPU(u32 readPtr) { // We can't wrap around while the GPU is working on the data. // This should be very rare due to the reset in SyncGPU. - SyncGPU(SYNC_GPU_WRAPAROUND); + SyncGPU(SyncGPUReason::Wraparound); if (!s_gpu_mainloop.IsRunning()) { // GPU is shutting down, so the next asserts may fail diff --git a/Source/Core/VideoCommon/Fifo.h b/Source/Core/VideoCommon/Fifo.h index d2172df655..ffe90c3ed9 100644 --- a/Source/Core/VideoCommon/Fifo.h +++ b/Source/Core/VideoCommon/Fifo.h @@ -20,15 +20,15 @@ void UpdateWantDeterminism(bool want); bool UseDeterministicGPUThread(); // Used for diagnostics. -enum SyncGPUReason +enum class SyncGPUReason { - SYNC_GPU_OTHER, - SYNC_GPU_WRAPAROUND, - SYNC_GPU_EFB_POKE, - SYNC_GPU_PERFQUERY, - SYNC_GPU_BBOX, - SYNC_GPU_SWAP, - SYNC_GPU_AUX_SPACE, + Other, + Wraparound, + EFBPoke, + PerfQuery, + BBox, + Swap, + AuxSpace, }; // In deterministic GPU thread mode this waits for the GPU to be done with pending work. void SyncGPU(SyncGPUReason reason, bool may_move_read_ptr = true); diff --git a/Source/Core/VideoCommon/MainBase.cpp b/Source/Core/VideoCommon/MainBase.cpp index 3b0eb8bf0a..5323651708 100644 --- a/Source/Core/VideoCommon/MainBase.cpp +++ b/Source/Core/VideoCommon/MainBase.cpp @@ -51,7 +51,7 @@ void VideoBackendBase::Video_BeginField(u32 xfbAddr, u32 fbWidth, u32 fbStride, { if (m_initialized && g_ActiveConfig.bUseXFB && g_renderer) { - Fifo::SyncGPU(Fifo::SYNC_GPU_SWAP); + Fifo::SyncGPU(Fifo::SyncGPUReason::Swap); AsyncRequests::Event e; e.time = 0; @@ -106,7 +106,7 @@ u32 VideoBackendBase::Video_GetQueryResult(PerfQueryType type) return 0; } - Fifo::SyncGPU(Fifo::SYNC_GPU_PERFQUERY); + Fifo::SyncGPU(Fifo::SyncGPUReason::PerfQuery); AsyncRequests::Event e; e.time = 0; @@ -133,7 +133,7 @@ u16 VideoBackendBase::Video_GetBoundingBox(int index) return 0; } - Fifo::SyncGPU(Fifo::SYNC_GPU_BBOX); + Fifo::SyncGPU(Fifo::SyncGPUReason::BBox); AsyncRequests::Event e; u16 result;