From: Joost VandeVondele Date: Thu, 5 Jul 2018 16:33:18 +0000 (+0200) Subject: Remove offset in thread redistribution scheme. X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=d2752fdc15f94b2eeb0a91533cc79c100738c2af Remove offset in thread redistribution scheme. doesn't have a benefit. passed STC (8 threads): LLR: 2.95 (-2.94,2.94) [-3.00,1.00] Total: 19574 W: 4028 L: 3904 D: 11642 http://tests.stockfishchess.org/tests/view/5b3e48950ebc5902b9fff080 passed LTC (8 threads): LLR: 2.96 (-2.94,2.94) [-3.00,1.00] Total: 21293 W: 3626 L: 3506 D: 14161 http://tests.stockfishchess.org/tests/view/5b3eefd60ebc5902b9fffa81 Closes https://github.com/official-stockfish/Stockfish/pull/1667 No functional change single threaded. --- diff --git a/src/search.cpp b/src/search.cpp index 97ad9912..7cb53fbf 100644 --- a/src/search.cpp +++ b/src/search.cpp @@ -331,7 +331,7 @@ void Thread::search() { if (idx > 0) { int i = (idx - 1) % 20; - if (((rootDepth / ONE_PLY + rootPos.game_ply() + SkipPhase[i]) / SkipSize[i]) % 2) + if (((rootDepth / ONE_PLY + SkipPhase[i]) / SkipSize[i]) % 2) continue; // Retry with an incremented rootDepth }