X-Git-Url: https://git.sesse.net/?p=stockfish;a=blobdiff_plain;f=src%2Fmovepick.cpp;h=17516603d207920c3bf362ff7510192f171787cf;hp=a562662e359e514d878b17879dcd18395d940553;hb=1e586288ca4a0ae567c8d9b9b77e804f25fe6719;hpb=0855d93de8cf7c68af616a2d5f384a7632470895 diff --git a/src/movepick.cpp b/src/movepick.cpp index a562662e..1ec56c2f 100644 --- a/src/movepick.cpp +++ b/src/movepick.cpp @@ -1,14 +1,14 @@ /* Stockfish, a UCI chess playing engine derived from Glaurung 2.1 Copyright (C) 2004-2008 Tord Romstad (Glaurung author) - Copyright (C) 2008-2009 Marco Costalba + Copyright (C) 2008-2015 Marco Costalba, Joona Kiiski, Tord Romstad + Copyright (C) 2015-2016 Marco Costalba, Joona Kiiski, Gary Linscott, Tord Romstad Stockfish is free software: you can redistribute it and/or modify it under the terms of the GNU General Public License as published by the Free Software Foundation, either version 3 of the License, or (at your option) any later version. - Stockfish is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the @@ -18,357 +18,334 @@ along with this program. If not, see . */ - -//// -//// Includes -//// - #include -#include "history.h" -#include "movegen.h" #include "movepick.h" -#include "search.h" -#include "value.h" - - -//// -//// Local definitions -//// +#include "thread.h" namespace { - enum MovegenPhase { - PH_TT_MOVES, // Transposition table move and mate killer - PH_GOOD_CAPTURES, // Queen promotions and captures with SEE values >= 0 - PH_KILLERS, // Killer moves from the current ply - PH_NONCAPTURES, // Non-captures and underpromotions - PH_BAD_CAPTURES, // Queen promotions and captures with SEE values < 0 - PH_EVASIONS, // Check evasions - PH_QCAPTURES, // Captures in quiescence search - PH_QCHECKS, // Non-capture checks in quiescence search - PH_STOP + enum Stages { + MAIN_SEARCH, CAPTURES_INIT, GOOD_CAPTURES, KILLERS, COUNTERMOVE, QUIET_INIT, QUIET, BAD_CAPTURES, + EVASION, EVASIONS_INIT, ALL_EVASIONS, + PROBCUT, PROBCUT_INIT, PROBCUT_CAPTURES, + QSEARCH_WITH_CHECKS, QCAPTURES_1_INIT, QCAPTURES_1, QCHECKS, + QSEARCH_NO_CHECKS, QCAPTURES_2_INIT, QCAPTURES_2, + QSEARCH_RECAPTURES, QRECAPTURES }; - CACHE_LINE_ALIGNMENT - const uint8_t MainSearchPhaseTable[] = { PH_TT_MOVES, PH_GOOD_CAPTURES, PH_KILLERS, PH_NONCAPTURES, PH_BAD_CAPTURES, PH_STOP}; - const uint8_t EvasionsPhaseTable[] = { PH_EVASIONS, PH_STOP}; - const uint8_t QsearchWithChecksPhaseTable[] = { PH_TT_MOVES, PH_QCAPTURES, PH_QCHECKS, PH_STOP}; - const uint8_t QsearchWithoutChecksPhaseTable[] = { PH_TT_MOVES, PH_QCAPTURES, PH_STOP}; -} + // Our insertion sort, which is guaranteed to be stable, as it should be + void insertion_sort(ExtMove* begin, ExtMove* end) + { + ExtMove tmp, *p, *q; + + for (p = begin + 1; p < end; ++p) + { + tmp = *p; + for (q = p; q != begin && *(q-1) < tmp; --q) + *q = *(q-1); + *q = tmp; + } + } + // pick_best() finds the best move in the range (begin, end) and moves it to + // the front. It's faster than sorting all the moves in advance when there + // are few moves, e.g., the possible captures. + Move pick_best(ExtMove* begin, ExtMove* end) + { + std::swap(*begin, *std::max_element(begin, end)); + return *begin; + } -//// -//// Functions -//// +} // namespace -/// Constructor for the MovePicker class. Apart from the position for which -/// it is asked to pick legal moves, MovePicker also wants some information -/// to help it to return the presumably good moves first, to decide which +/// Constructors of the MovePicker class. As arguments we pass information +/// to help it to return the (presumably) good moves first, to decide which /// moves to return (in the quiescence search, for instance, we only want to -/// search captures, promotions and some checks) and about how important good -/// move ordering is at the current node. +/// search captures, promotions, and some checks) and how important good move +/// ordering is at the current node. -MovePicker::MovePicker(const Position& p, Move ttm, Depth d, - const History& h, SearchStack* ss) : pos(p), H(h) { - int searchTT = ttm; - ttMoves[0].move = ttm; - finished = false; - lastBadCapture = badCaptures; +MovePicker::MovePicker(const Position& p, Move ttm, Depth d, Search::Stack* s) + : pos(p), ss(s), depth(d) { - if (ss) - { - ttMoves[1].move = (ss->mateKiller == ttm)? MOVE_NONE : ss->mateKiller; - searchTT |= ttMoves[1].move; - killers[0].move = ss->killers[0]; - killers[1].move = ss->killers[1]; - } else - ttMoves[1].move = killers[0].move = killers[1].move = MOVE_NONE; - - Color us = pos.side_to_move(); - - dc = p.discovered_check_candidates(us); - pinned = p.pinned_pieces(us); - - if (p.is_check()) - phasePtr = EvasionsPhaseTable; - else if (d > Depth(0)) - phasePtr = MainSearchPhaseTable + !searchTT; - else if (d == Depth(0)) - phasePtr = QsearchWithChecksPhaseTable + !searchTT; - else - phasePtr = QsearchWithoutChecksPhaseTable + !searchTT; + assert(d > DEPTH_ZERO); + + Square prevSq = to_sq((ss-1)->currentMove); + countermove = pos.this_thread()->counterMoves[pos.piece_on(prevSq)][prevSq]; - phasePtr--; - go_next_phase(); + stage = pos.checkers() ? EVASION : MAIN_SEARCH; + ttMove = ttm && pos.pseudo_legal(ttm) ? ttm : MOVE_NONE; + stage += (ttMove == MOVE_NONE); } +MovePicker::MovePicker(const Position& p, Move ttm, Depth d, Square s) + : pos(p) { -/// MovePicker::go_next_phase() generates, scores and sorts the next bunch -/// of moves when there are no more moves to try for the current phase. + assert(d <= DEPTH_ZERO); -void MovePicker::go_next_phase() { + if (pos.checkers()) + stage = EVASION; - curMove = moves; - phase = *(++phasePtr); - switch (phase) { + else if (d > DEPTH_QS_NO_CHECKS) + stage = QSEARCH_WITH_CHECKS; - case PH_TT_MOVES: - curMove = ttMoves; - lastMove = curMove + 2; - return; + else if (d > DEPTH_QS_RECAPTURES) + stage = QSEARCH_NO_CHECKS; - case PH_GOOD_CAPTURES: - lastMove = generate_captures(pos, moves); - score_captures(); + else + { + stage = QSEARCH_RECAPTURES; + recaptureSquare = s; return; + } - case PH_KILLERS: - curMove = killers; - lastMove = curMove + 2; - return; + ttMove = ttm && pos.pseudo_legal(ttm) ? ttm : MOVE_NONE; + stage += (ttMove == MOVE_NONE); +} - case PH_NONCAPTURES: - lastMove = generate_noncaptures(pos, moves); - score_noncaptures(); - sort_moves(moves, lastMove); - return; +MovePicker::MovePicker(const Position& p, Move ttm, Value th) + : pos(p), threshold(th) { - case PH_BAD_CAPTURES: - // Bad captures SEE value is already calculated so just sort them - // to get SEE move ordering. - curMove = badCaptures; - lastMove = lastBadCapture; - return; + assert(!pos.checkers()); - case PH_EVASIONS: - assert(pos.is_check()); - lastMove = generate_evasions(pos, moves); - score_evasions(); - return; + stage = PROBCUT; - case PH_QCAPTURES: - lastMove = generate_captures(pos, moves); - score_captures(); - return; + // In ProbCut we generate captures with SEE higher than the given threshold + ttMove = ttm + && pos.pseudo_legal(ttm) + && pos.capture(ttm) + && pos.see(ttm) > threshold ? ttm : MOVE_NONE; - case PH_QCHECKS: - // Perhaps we should order moves move here? FIXME - lastMove = generate_non_capture_checks(pos, moves, dc); - return; + stage += (ttMove == MOVE_NONE); +} - case PH_STOP: - lastMove = curMove + 1; // hack to be friendly for get_next_move() - return; - default: - assert(false); - return; - } +/// score() assigns a numerical value to each move in a move list. The moves with +/// highest values will be picked first. +template<> +void MovePicker::score() { + // Winning and equal captures in the main search are ordered by MVV, preferring + // captures near our home rank. Surprisingly, this appears to perform slightly + // better than SEE-based move ordering: exchanging big pieces before capturing + // a hanging piece probably helps to reduce the subtree size. + // In the main search we want to push captures with negative SEE values to the + // badCaptures[] array, but instead of doing it now we delay until the move + // has been picked up, saving some SEE calls in case we get a cutoff. + for (auto& m : *this) + m.value = PieceValue[MG][pos.piece_on(to_sq(m))] + - Value(200 * relative_rank(pos.side_to_move(), to_sq(m))); } +template<> +void MovePicker::score() { -/// MovePicker::score_captures(), MovePicker::score_noncaptures() and -/// MovePicker::score_evasions() assign a numerical move ordering score -/// to each move in a move list. The moves with highest scores will be -/// picked first by get_next_move(). - -void MovePicker::score_captures() { - // Winning and equal captures in the main search are ordered by MVV/LVA. - // Suprisingly, this appears to perform slightly better than SEE based - // move ordering. The reason is probably that in a position with a winning - // capture, capturing a more valuable (but sufficiently defended) piece - // first usually doesn't hurt. The opponent will have to recapture, and - // the hanging piece will still be hanging (except in the unusual cases - // where it is possible to recapture with the hanging piece). Exchanging - // big pieces before capturing a hanging piece probably helps to reduce - // the subtree size. - // In main search we want to push captures with negative SEE values to - // badCaptures[] array, but instead of doing it now we delay till when - // the move has been picked up in pick_move_from_list(), this way we save - // some SEE calls in case we get a cutoff (idea from Pablo Vazquez). - Move m; - - // Use MVV/LVA ordering - for (MoveStack* cur = moves; cur != lastMove; cur++) - { - m = cur->move; - if (move_is_promotion(m)) - cur->score = QueenValueMidgame; - else - cur->score = pos.midgame_value_of_piece_on(move_to(m)) - - pos.type_of_piece_on(move_from(m)); - } -} + const HistoryStats& history = pos.this_thread()->history; + const FromToStats& fromTo = pos.this_thread()->fromTo; -void MovePicker::score_noncaptures() { - // First score by history, when no history is available then use - // piece/square tables values. This seems to be better then a - // random choice when we don't have an history for any move. - Move m; - Piece piece; - Square from, to; - int hs; + const CounterMoveStats* cm = (ss-1)->counterMoves; + const CounterMoveStats* fm = (ss-2)->counterMoves; + const CounterMoveStats* f2 = (ss-4)->counterMoves; - for (MoveStack* cur = moves; cur != lastMove; cur++) - { - m = cur->move; - from = move_from(m); - to = move_to(m); - piece = pos.piece_on(from); - hs = H.move_ordering_score(piece, to); - - // Ensure history is always preferred to pst - if (hs > 0) - hs += 1000; - - // pst based scoring - cur->score = hs + pos.pst_delta(piece, from, to); - } -} + Color c = pos.side_to_move(); -void MovePicker::score_evasions() { - // Always try ttMove as first. Then try good captures ordered - // by MVV/LVA, then non-captures if destination square is not - // under attack, ordered by history value, and at the end - // bad-captures and non-captures with a negative SEE. This - // last group is ordered by the SEE score. - Move m; - int seeScore; + for (auto& m : *this) + m.value = history[pos.moved_piece(m)][to_sq(m)] + + (cm ? (*cm)[pos.moved_piece(m)][to_sq(m)] : VALUE_ZERO) + + (fm ? (*fm)[pos.moved_piece(m)][to_sq(m)] : VALUE_ZERO) + + (f2 ? (*f2)[pos.moved_piece(m)][to_sq(m)] : VALUE_ZERO) + + fromTo.get(c, m); +} - for (MoveStack* cur = moves; cur != lastMove; cur++) - { - m = cur->move; - if (m == ttMoves[0].move) - cur->score = 2 * HistoryMax; - else if ((seeScore = pos.see_sign(m)) < 0) - cur->score = seeScore; - else if (pos.move_is_capture(m)) - cur->score = pos.midgame_value_of_piece_on(move_to(m)) - - pos.type_of_piece_on(move_from(m)) + HistoryMax; +template<> +void MovePicker::score() { + // Try captures ordered by MVV/LVA, then non-captures ordered by history value + const HistoryStats& history = pos.this_thread()->history; + const FromToStats& fromTo = pos.this_thread()->fromTo; + Color c = pos.side_to_move(); + + for (auto& m : *this) + if (pos.capture(m)) + m.value = PieceValue[MG][pos.piece_on(to_sq(m))] + - Value(type_of(pos.moved_piece(m))) + HistoryStats::Max; else - cur->score = H.move_ordering_score(pos.piece_on(move_from(m)), move_to(m)); - } + m.value = history[pos.moved_piece(m)][to_sq(m)] + fromTo.get(c, m); } -/// MovePicker::get_next_move() is the most important method of the MovePicker -/// class. It returns a new legal move every time it is called, until there -/// are no more moves left. -/// It picks the move with the biggest score from a list of generated moves taking -/// care not to return the tt move if has already been searched previously. -Move MovePicker::get_next_move() { +/// next_move() is the most important method of the MovePicker class. It returns +/// a new pseudo legal move every time it is called, until there are no more moves +/// left. It picks the move with the biggest value from a list of generated moves +/// taking care not to return the ttMove if it has already been searched. - assert(!pos.is_check() || *phasePtr == PH_EVASIONS || *phasePtr == PH_STOP); - assert( pos.is_check() || *phasePtr != PH_EVASIONS); +Move MovePicker::next_move() { Move move; - while (true) - { - while (curMove != lastMove) - { - switch (phase) { + switch (stage) { - case PH_TT_MOVES: - move = (curMove++)->move; - if ( move != MOVE_NONE - && move_is_legal(pos, move, pinned)) - return move; - break; - - case PH_GOOD_CAPTURES: - move = pick_best(curMove++, lastMove).move; - if ( move != ttMoves[0].move - && move != ttMoves[1].move - && pos.pl_move_is_legal(move, pinned)) - { - // Check for a non negative SEE now - int seeValue = pos.see_sign(move); - if (seeValue >= 0) - return move; - - // Losing capture, move it to the badCaptures[] array, note - // that move has now been already checked for legality. - assert(int(lastBadCapture - badCaptures) < 63); - lastBadCapture->move = move; - lastBadCapture->score = seeValue; - lastBadCapture++; - } - break; - - case PH_KILLERS: - move = (curMove++)->move; - if ( move != MOVE_NONE - && move != ttMoves[0].move - && move != ttMoves[1].move - && move_is_legal(pos, move, pinned) - && !pos.move_is_capture(move)) - return move; - break; - - case PH_NONCAPTURES: - move = (curMove++)->move; - if ( move != ttMoves[0].move - && move != ttMoves[1].move - && move != killers[0].move - && move != killers[1].move - && pos.pl_move_is_legal(move, pinned)) - return move; - break; + case MAIN_SEARCH: case EVASION: case QSEARCH_WITH_CHECKS: + case QSEARCH_NO_CHECKS: case PROBCUT: + ++stage; + return ttMove; - case PH_BAD_CAPTURES: - move = pick_best(curMove++, lastMove).move; - return move; + case CAPTURES_INIT: + endBadCaptures = cur = moves; + endMoves = generate(pos, cur); + score(); + ++stage; - case PH_EVASIONS: - move = pick_best(curMove++, lastMove).move; - if (pos.pl_move_is_legal(move, pinned)) + case GOOD_CAPTURES: + while (cur < endMoves) + { + move = pick_best(cur++, endMoves); + if (move != ttMove) + { + if (pos.see_sign(move) >= VALUE_ZERO) return move; - break; - case PH_QCAPTURES: - move = pick_best(curMove++, lastMove).move; - if ( move != ttMoves[0].move - && pos.pl_move_is_legal(move, pinned)) - return move; - break; + // Losing capture, move it to the beginning of the array + *endBadCaptures++ = move; + } + } - case PH_QCHECKS: - move = (curMove++)->move; - if ( move != ttMoves[0].move - && pos.pl_move_is_legal(move, pinned)) - return move; - break; + ++stage; + move = ss->killers[0]; // First killer move + if ( move != MOVE_NONE + && move != ttMove + && pos.pseudo_legal(move) + && !pos.capture(move)) + return move; + + case KILLERS: + ++stage; + move = ss->killers[1]; // Second killer move + if ( move != MOVE_NONE + && move != ttMove + && pos.pseudo_legal(move) + && !pos.capture(move)) + return move; + + case COUNTERMOVE: + ++stage; + move = countermove; + if ( move != MOVE_NONE + && move != ttMove + && move != ss->killers[0] + && move != ss->killers[1] + && pos.pseudo_legal(move) + && !pos.capture(move)) + return move; + + case QUIET_INIT: + cur = endBadCaptures; + endMoves = generate(pos, cur); + score(); + if (depth < 3 * ONE_PLY) + { + ExtMove* goodQuiet = std::partition(cur, endMoves, [](const ExtMove& m) + { return m.value > VALUE_ZERO; }); + insertion_sort(cur, goodQuiet); + } else + insertion_sort(cur, endMoves); + ++stage; + + case QUIET: + while (cur < endMoves) + { + move = *cur++; + if ( move != ttMove + && move != ss->killers[0] + && move != ss->killers[1] + && move != countermove) + return move; + } + ++stage; + cur = moves; // Point to beginning of bad captures + + case BAD_CAPTURES: + if (cur < endBadCaptures) + return *cur++; + break; + + case EVASIONS_INIT: + cur = moves; + endMoves = generate(pos, cur); + if (endMoves - cur - (ttMove != MOVE_NONE) > 1) + score(); + ++stage; + + case ALL_EVASIONS: + while (cur < endMoves) + { + move = pick_best(cur++, endMoves); + if (move != ttMove) + return move; + } + break; - case PH_STOP: - return MOVE_NONE; + case PROBCUT_INIT: + cur = moves; + endMoves = generate(pos, cur); + score(); + ++stage; - default: - assert(false); - break; - } + case PROBCUT_CAPTURES: + while (cur < endMoves) + { + move = pick_best(cur++, endMoves); + if ( move != ttMove + && pos.see(move) > threshold) + return move; } - go_next_phase(); - } -} + break; -/// A variant of get_next_move() which takes a lock as a parameter, used to -/// prevent multiple threads from picking the same move at a split point. + case QCAPTURES_1_INIT: case QCAPTURES_2_INIT: + cur = moves; + endMoves = generate(pos, cur); + score(); + ++stage; -Move MovePicker::get_next_move(Lock &lock) { + case QCAPTURES_1: case QCAPTURES_2: + while (cur < endMoves) + { + move = pick_best(cur++, endMoves); + if (move != ttMove) + return move; + } + if (stage == QCAPTURES_2) + break; + cur = moves; + endMoves = generate(pos, cur); + ++stage; + + case QCHECKS: + while (cur < endMoves) + { + move = cur++->move; + if (move != ttMove) + return move; + } + break; + + case QSEARCH_RECAPTURES: + cur = moves; + endMoves = generate(pos, cur); + score(); + ++stage; - lock_grab(&lock); - if (finished) - { - lock_release(&lock); - return MOVE_NONE; - } - Move m = get_next_move(); - if (m == MOVE_NONE) - finished = true; + case QRECAPTURES: + while (cur < endMoves) + { + move = pick_best(cur++, endMoves); + if (to_sq(move) == recaptureSquare) + return move; + } + break; + + default: + assert(false); + } - lock_release(&lock); - return m; + return MOVE_NONE; }