Get correct excluded moves for split nodes
authorHongzhi Cheng <hongzhi_cheng@yahoo.com>
Mon, 18 Nov 2013 15:41:19 +0000 (16:41 +0100)
committerMarco Costalba <mcostalba@gmail.com>
Mon, 18 Nov 2013 15:41:49 +0000 (16:41 +0100)
Tested setting FakeSplit to true and running

./stockfish bench 128 2

There is a different signature with and without
the patch so it affects functionality but
only in SMP case.

bench: 8331357

src/search.cpp

index f1ab80f29294e20401fb93e9736b3b2b679668de..5ae2cf69f54dad439df37f5130245289b1fd0562 100644 (file)
@@ -499,6 +499,7 @@ namespace {
     // Step 1. Initialize node
     Thread* thisThread = pos.this_thread();
     inCheck = pos.checkers();
+    excludedMove = ss->excludedMove;
 
     if (SpNode)
     {
@@ -507,7 +508,7 @@ namespace {
         threatMove = splitPoint->threatMove;
         bestValue  = splitPoint->bestValue;
         tte = NULL;
-        ttMove = excludedMove = MOVE_NONE;
+        ttMove = MOVE_NONE;
         ttValue = VALUE_NONE;
 
         assert(splitPoint->bestValue > -VALUE_INFINITE && splitPoint->moveCount > 0);
@@ -547,7 +548,6 @@ namespace {
     // Step 4. Transposition table lookup
     // We don't want the score of a partial search to overwrite a previous full search
     // TT value, so we use a different position key in case of an excluded move.
-    excludedMove = ss->excludedMove;
     posKey = excludedMove ? pos.exclusion_key() : pos.key();
     tte = TT.probe(posKey);
     ttMove = RootNode ? RootMoves[PVIdx].pv[0] : tte ? tte->move() : MOVE_NONE;