From: protonspring <mike@whiteley.org>
Date: Mon, 5 Feb 2018 16:27:59 +0000 (+0100)
Subject:     Don’t score and sort all captures in RECAPTURES stage.
X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=414a3e6ee3ab38215f9eba69939773ba62a9766e

Don’t score and sort all captures in RECAPTURES stage.

    For these recaptures, we’re are only considering those captures
    that recapture the recapture square (small portion of all the
    captures). Therefore, scoring all of the captures and pick_besting
    out of the whole group is not necessary.

    STC
    LLR: 2.96 (-2.94,2.94) [-3.00,1.00]
    Total: 85583 W: 18978 L: 18983 D: 47622
    http://tests.stockfishchess.org/tests/view/5a717faa0ebc590f2c86e9a7

    LTC
    LLR: 2.96 (-2.94,2.94) [-3.00,1.00]
    Total: 20231 W: 3533 L: 3411 D: 13287
    http://tests.stockfishchess.org/tests/view/5a73ad330ebc5902971a96ba

    Bench: 5023593
---

diff --git a/src/movepick.cpp b/src/movepick.cpp
index 9882960d..74da3205 100644
--- a/src/movepick.cpp
+++ b/src/movepick.cpp
@@ -307,14 +307,13 @@ Move MovePicker::next_move(bool skipQuiets) {
   case QSEARCH_RECAPTURES:
       cur = moves;
       endMoves = generate<CAPTURES>(pos, cur);
-      score<CAPTURES>();
       ++stage;
       /* fallthrough */
 
   case QRECAPTURES:
       while (cur < endMoves)
       {
-          move = pick_best(cur++, endMoves);
+          move = *cur++;
           if (to_sq(move) == recaptureSquare)
               return move;
       }