Remove unneeded & incorrect check.

the removed line is not needed, since with the conditions on SE, eval
equals ttValue (except inCheck), which must be larger than beta if the second condition
is true.

The removed line is also incorrect as eval might be VALUE_NONE at this
location if inCheck. This removal addresses part of https://github.com/official-stockfish/Stockfish/pull/2406#issuecomment-552642608

No functional change.
This commit is contained in:
Joost VandeVondele
2019-11-12 16:12:09 +01:00
committed by Stéphane Nicolet
parent 6a6fc28551
commit 0256416bb7

View File

@@ -1033,8 +1033,7 @@ moves_loop: // When in check, search starts from here
// search without the ttMove. So we assume this expected Cut-node is not singular,
// that multiple moves fail high, and we can prune the whole subtree by returning
// a soft bound.
else if ( eval >= beta
&& singularBeta >= beta)
else if (singularBeta >= beta)
return singularBeta;
}