From 222df615c1bbce15290d33f3f38d401a6df1743a Mon Sep 17 00:00:00 2001 From: "Robert Nurnberg @ elitebook" Date: Mon, 25 Aug 2025 11:09:39 +0200 Subject: [PATCH] revert #6259 The recent commit af181d9 was merged as a simplification, but unfortunately hurts mate finding efficiency. https://github.com/vondele/matetrack/blob/69f5c5e8627c163a6a8480b869ee09bc44dc44d4/matetrack1000000.csv#L4075-L4076 So this PR proposes to revert it, while adding a comment in the code for future reference. closes https://github.com/official-stockfish/Stockfish/pull/6269 Bench: 2566711 --- src/movepick.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/movepick.cpp b/src/movepick.cpp index 4b01beb67..b4523463f 100644 --- a/src/movepick.cpp +++ b/src/movepick.cpp @@ -115,7 +115,9 @@ MovePicker::MovePicker(const Position& p, Move ttm, int th, const CapturePieceTo threshold(th) { assert(!pos.checkers()); - stage = PROBCUT_TT + !(ttm && pos.capture_stage(ttm) && pos.pseudo_legal(ttm)); + // Removing the SEE check passes as simplification, but hurts mate finding + stage = PROBCUT_TT + + !(ttm && pos.capture_stage(ttm) && pos.pseudo_legal(ttm) && pos.see_ge(ttm, threshold)); } // Assigns a numerical value to each move in a list, used for sorting.