X-Git-Url: https://git.sesse.net/?p=stockfish;a=blobdiff_plain;f=src%2Fsearch.cpp;h=0e68178147c7b2cbba34845c89e57218e6ea9b86;hp=267f5f8f4c64c87fbacc15f9184a16242dbcd394;hb=289a767ab3a683a6f2aed74b738ceb87bb632466;hpb=817ca1820b59152ef511ed8cdabe9116c225a722 diff --git a/src/search.cpp b/src/search.cpp index 267f5f8f..0e681781 100644 --- a/src/search.cpp +++ b/src/search.cpp @@ -643,10 +643,10 @@ namespace { && !ss->skipNullMove && depth < 4 * ONE_PLY && !inCheck - && eval - futility_margin(depth, (ss-1)->futMc) >= beta + && eval - futility_margin(depth, (ss-1)->futilityMoveCount) >= beta && abs(beta) < VALUE_MATE_IN_MAX_PLY && pos.non_pawn_material(pos.side_to_move())) - return eval - futility_margin(depth, (ss-1)->futMc); + return eval - futility_margin(depth, (ss-1)->futilityMoveCount); // Step 8. Null move search with verification search (is omitted in PV nodes) if ( !PvNode @@ -806,6 +806,7 @@ split_point_start: // At split points actual search starts from here << " currmovenumber " << moveCount + PVIdx << sync_endl; } + ss->futilityMoveCount = 0; ext = DEPTH_ZERO; captureOrPromotion = pos.is_capture_or_promotion(move); givesCheck = pos.move_gives_check(move, ci); @@ -852,8 +853,6 @@ split_point_start: // At split points actual search starts from here // Update current move (this must be done after singular extension search) newDepth = depth - ONE_PLY + ext; - ss->futMc = 0; - // Step 13. Futility pruning (is omitted in PV nodes) if ( !PvNode && !captureOrPromotion @@ -876,8 +875,6 @@ split_point_start: // At split points actual search starts from here // Value based pruning // We illogically ignore reduction condition depth >= 3*ONE_PLY for predicted depth, // but fixing this made program slightly weaker. - ss->futMc = moveCount; - Depth predictedDepth = newDepth - reduction(depth, moveCount); futilityValue = ss->staticEval + ss->evalMargin + futility_margin(predictedDepth, moveCount) + Gain[pos.piece_moved(move)][to_sq(move)]; @@ -904,6 +901,10 @@ split_point_start: // At split points actual search starts from here continue; } + + // We have not pruned the move that will be searched, but remember how + // far in the move list we are to be more aggressive in the child node. + ss->futilityMoveCount = moveCount; } // Check for legality only before to do the move