Don't uselessy share rootDepth

It is not needed becuase the only case is a real special
one (bench on depth with many threads) and can be easily
rewritten to avoid sharing rootDepth.

Verified with ThreadSanitizer.

No functional change.

Closes #1159

diff --git a/src/search.cpp b/src/search.cpp
index e6c3cd701485947c1c174b98f037661dca03409e..09e3c4fa979efd0620d021e14daacbb51ea8034c 100644 (file)
--- a/src/search.cpp
+++ b/src/search.cpp
@@ -360,9 +360,9 @@ void Thread::search() {
   multiPV = std::min(multiPV, rootMoves.size());
 
   // Iterative deepening loop until requested to stop or the target depth is reached
-  while (   (rootDepth = rootDepth + ONE_PLY) < DEPTH_MAX
+  while (   (rootDepth += ONE_PLY) < DEPTH_MAX
          && !Signals.stop
-         && (!Limits.depth || Threads.main()->rootDepth / ONE_PLY <= Limits.depth))
+         && !(Limits.depth && mainThread && rootDepth / ONE_PLY > Limits.depth))
   {
       // Distribute search depths across the threads
       if (idx)

diff --git a/src/thread.h b/src/thread.h
index c254047c23f4ce90a04415bcc97090f3c57aae0b..dc0c51c28d5c2401e415ccde5ce361b43061a3b9 100644 (file)
--- a/src/thread.h
+++ b/src/thread.h
@@ -65,7 +65,7 @@ public:
 
   Position rootPos;
   Search::RootMoves rootMoves;
-  std::atomic<Depth> rootDepth;
+  Depth rootDepth;
   Depth completedDepth;
   CounterMoveStat counterMoves;
   ButterflyHistory history;