mirror of
https://github.com/HChaZZY/Stockfish.git
synced 2025-12-22 01:56:58 +08:00
Remove dead code in search
We can never have bestValue == -VALUE_INFINITE at the end of move loop because if no legal move exists we detect it with previous condition on !moveCount, if a legal move exists we never prune it due to futility pruning condition: bestValue > VALUE_MATED_IN_MAX_PLY So this code never executes, as I have also verified directly. Issue reported by Joona. No functional change.
This commit is contained in:
@@ -1017,10 +1017,6 @@ moves_loop: // When in check and at SpNode search starts from here
|
|||||||
return excludedMove ? alpha
|
return excludedMove ? alpha
|
||||||
: inCheck ? mated_in(ss->ply) : DrawValue[pos.side_to_move()];
|
: inCheck ? mated_in(ss->ply) : DrawValue[pos.side_to_move()];
|
||||||
|
|
||||||
// If we have pruned all the moves without searching return a fail-low score
|
|
||||||
if (bestValue == -VALUE_INFINITE)
|
|
||||||
bestValue = alpha;
|
|
||||||
|
|
||||||
TT.store(posKey, value_to_tt(bestValue, ss->ply),
|
TT.store(posKey, value_to_tt(bestValue, ss->ply),
|
||||||
bestValue >= beta ? BOUND_LOWER :
|
bestValue >= beta ? BOUND_LOWER :
|
||||||
PvNode && bestMove ? BOUND_EXACT : BOUND_UPPER,
|
PvNode && bestMove ? BOUND_EXACT : BOUND_UPPER,
|
||||||
|
|||||||
Reference in New Issue
Block a user