]> git.sesse.net Git - stockfish/commitdiff
Omit two unneeded tests
authorttruscott <ttruscott@users.noreply.github.com>
Fri, 25 Aug 2023 19:47:52 +0000 (15:47 -0400)
committerStéphane Nicolet <cassio@free.fr>
Sun, 3 Sep 2023 06:07:59 +0000 (08:07 +0200)
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

src/search.cpp

index d9b41cb35c4f87ee900106caf7ea0900962e239c..18e4aa56df2cfa60157cef86e570999076471f93 100644 (file)
@@ -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))
     {