deepcrayon
/
stockfish
1
0
Fork 0
Code Releases Activity

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
pull/1155/merge
Marco Costalba 2017-07-02 03:10:50 -07:00 committed by Joona Kiiski
parent 01b6cdb76b
commit 802fca6fdd
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/search.cpp
View File

@ -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)

2
src/thread.h
View File

@ -65,7 +65,7 @@ public:
Position rootPos;
Search::RootMoves rootMoves;
std::atomic<Depth> rootDepth;
Depth rootDepth;
Depth completedDepth;
CounterMoveStat counterMoves;
ButterflyHistory history;