Data race fix

There's a data race involving threadCount, so it is now an AtomicInt object. Also removed unused extern variables. Bug chromium:778254 chromium:776621 Change-Id: I93174e8f4e5e48f26414ba9c81bb9a2c317dbf73 Reviewed-on: https://swiftshader-review.googlesource.com/13290Tested-by: Alexis Hétu <sugoi@google.com> Reviewed-by: Nicolas Capens <nicolascapens@google.com>

Data race fix
f2520e19 · Alexis Hetu · Alexis Hétu · 74b91b5a · f2520e19 · f2520e19
Commit f2520e19 authored Oct 26, 2017 by Alexis Hetu Committed by Alexis Hétu Oct 26, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 5 deletions

Renderer.cpp src/Renderer/Renderer.cpp +2 -2

Renderer.hpp src/Renderer/Renderer.hpp +0 -3

No files found.
--- a/src/Renderer/Renderer.cpp
+++ b/src/Renderer/Renderer.cpp
@@ -61,8 +61,8 @@ namespace sw
 	extern bool precacheSetup;
 	extern bool precachePixel;

-	int batchSize = 128;
-	int threadCount = 1;
+	static const int batchSize = 128;
+	AtomicInt threadCount(1);
 	AtomicInt Renderer::unitCount(1);
 	AtomicInt Renderer::clusterCount(1);


--- a/src/Renderer/Renderer.hpp
+++ b/src/Renderer/Renderer.hpp
@@ -37,9 +37,6 @@ namespace sw
 	class Renderer;
 	struct Constants;

-	extern int batchSize;
-	extern int threadCount;
-
 	enum TranscendentalPrecision
 	{
 		APPROXIMATE,