From 0256416bb7b3ba7e96a487062104a9379c0c3a82 Mon Sep 17 00:00:00 2001 From: Joost VandeVondele Date: Tue, 12 Nov 2019 16:12:09 +0100 Subject: [PATCH] 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. --- src/search.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/search.cpp b/src/search.cpp index e976274f..26010108 100644 --- a/src/search.cpp +++ b/src/search.cpp @@ -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; } -- 2.39.2