commit 458736449e062d129998df506a802ad23dfafffd
author Nicolas Capens <capn@google.com> Wed Aug 28 09:54:20 2019 -0400
committer Nicolas Capens <nicolascapens@google.com> Wed Aug 28 16:56:31 2019 +0000
tree 1b725511484b4d6fed824289b2f9261fefa11b48
parent 05823febe04c828f8cd42c7f64f02a28195d02ba

Prettify atomic initialization

Change-Id: I1230607ebbf0e7dadf8603d2fde5e7a95441c713
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35628
Presubmit-Ready: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Ben Clayton <bclayton@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Tested-by: Nicolas Capens <nicolascapens@google.com>

src/Yarn/Scheduler.cpp

diff --git a/src/Yarn/Scheduler.cpp b/src/Yarn/Scheduler.cpp
index 827aa90..25b432e 100644
--- a/src/Yarn/Scheduler.cpp
+++ b/src/Yarn/Scheduler.cpp
@@ -99,7 +99,6 @@
 }
 
 Scheduler::Scheduler()
-    : nextEnqueueIndex(0)
 {
     for (size_t i = 0; i < spinningWorkers.size(); i++)
     {

src/Yarn/Scheduler.hpp

diff --git a/src/Yarn/Scheduler.hpp b/src/Yarn/Scheduler.hpp
index 992dade..7451698 100644
--- a/src/Yarn/Scheduler.hpp
+++ b/src/Yarn/Scheduler.hpp
@@ -294,7 +294,7 @@
 
     // TODO: Make this lot thread-safe so setWorkerThreadCount() can be called
     // during execution of tasks.
-    std::atomic<unsigned int> nextEnqueueIndex;
+    std::atomic<unsigned int> nextEnqueueIndex = { 0 };
     unsigned int numWorkerThreads = 0;
     std::array<Worker*, MaxWorkerThreads> workerThreads;