From: ttruscott <ttruscott@users.noreply.github.com>
Date: Fri, 25 Aug 2023 19:47:52 +0000 (-0400)
Subject: Omit two unneeded tests
X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=8cd5cbf6939d76b33a744f1379a6f84a4ac3a6cb

Omit two unneeded tests

These redundant tests were intended as a speed-up, but they do not seem
to provide any speed anymore.

STC: https://tests.stockfishchess.org/tests/view/64e9079c85e3e95030fd8259
LLR: 2.96 (-2.94,2.94) <-1.75,0.25>
Total: 134688 W: 34338 L: 34226 D: 66124
Ptnml(0-2): 426, 15122, 36124, 15258, 414

closes https://github.com/official-stockfish/Stockfish/pull/4767

No functional change
---

diff --git a/src/search.cpp b/src/search.cpp
index d9b41cb3..18e4aa56 100644
--- a/src/search.cpp
+++ b/src/search.cpp
@@ -518,7 +518,6 @@ namespace {
     // Check if we have an upcoming move that draws by repetition, or
     // if the opponent had an alternative move earlier to this position.
     if (   !rootNode
-        && pos.rule50_count() >= 3
         && alpha < VALUE_DRAW
         && pos.has_game_cycle(ss->ply))
     {
@@ -1398,7 +1397,6 @@ moves_loop: // When in check, search starts here
     // Check if we have an upcoming move that draws by repetition, or
     // if the opponent had an alternative move earlier to this position.
     if (   depth < 0
-        && pos.rule50_count() >= 3
         && alpha < VALUE_DRAW
         && pos.has_game_cycle(ss->ply))
     {