From f3efe47ff6c8b4acc42d14e97ab30926ae33ba6f Mon Sep 17 00:00:00 2001 From: wraitii Date: Sat, 5 Jun 2021 09:48:41 +0000 Subject: [PATCH] Default-init atomics in TaskManager::Impl std::atomic does not value-initialise the boolean. This caused windows tests to use way too much CPU and fail. Differential Revision: https://code.wildfiregames.com/D4089 This was SVN commit r25687. --- source/ps/TaskManager.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/source/ps/TaskManager.cpp b/source/ps/TaskManager.cpp index 7e34a9fd24..63f5f4472e 100644 --- a/source/ps/TaskManager.cpp +++ b/source/ps/TaskManager.cpp @@ -141,8 +141,8 @@ protected: // Back reference (keep this first). TaskManager& m_TaskManager; - std::atomic m_HasWork; - std::atomic m_HasLowPriorityWork; + std::atomic m_HasWork = false; + std::atomic m_HasLowPriorityWork = false; std::mutex m_GlobalMutex; std::mutex m_GlobalLowPriorityMutex; std::deque m_GlobalQueue;