X-Git-Url: https://git.sesse.net/?p=stockfish;a=blobdiff_plain;f=src%2Fmovepick.cpp;h=b0469a66abd5f08ce6facf8754ea1344f895e887;hp=63f77dbaebfc14570cb4ff56408348fa4b15adb0;hb=a952c6bc6d64ba4c9fe4f0952896a9fd6e63b4a9;hpb=a010d438a2e378a82943835d49a53dfaa15c95d1 diff --git a/src/movepick.cpp b/src/movepick.cpp index 63f77dba..b0469a66 100644 --- a/src/movepick.cpp +++ b/src/movepick.cpp @@ -90,16 +90,16 @@ MovePicker::MovePicker(const Position& p, Move ttm, Depth d, const History& h, if (p.is_check()) phasePtr = EvasionsPhaseTable; - else if (d > Depth(0)) + else if (d > DEPTH_ZERO) { // Consider sligtly negative captures as good if at low // depth and far from beta. - if (ss && ss->eval < beta - PawnValueMidgame && d < 3 * OnePly) + if (ss && ss->eval < beta - PawnValueMidgame && d < 3 * ONE_PLY) badCaptureThreshold = -PawnValueMidgame; phasePtr = MainSearchPhaseTable; } - else if (d == Depth(0)) + else if (d == DEPTH_ZERO) phasePtr = QsearchWithChecksPhaseTable; else { @@ -157,7 +157,7 @@ void MovePicker::go_next_phase() { case PH_EVASIONS: assert(pos.is_check()); lastMove = generate_evasions(pos, moves); - score_evasions_or_checks(); + score_evasions(); return; case PH_QCAPTURES: @@ -167,7 +167,6 @@ void MovePicker::go_next_phase() { case PH_QCHECKS: lastMove = generate_non_capture_checks(pos, moves); - score_evasions_or_checks(); return; case PH_STOP: @@ -221,7 +220,6 @@ void MovePicker::score_noncaptures() { Move m; Piece piece; Square from, to; - int hs; for (MoveStack* cur = moves; cur != lastMove; cur++) { @@ -229,18 +227,11 @@ void MovePicker::score_noncaptures() { from = move_from(m); to = move_to(m); piece = pos.piece_on(from); - hs = H.move_ordering_score(piece, to); - - // Ensure history has always highest priority - if (hs > 0) - hs += 10000; - - // Gain table based scoring - cur->score = hs + 16 * H.gain(piece, to); + cur->score = H.value(piece, to) + H.gain(piece, to); } } -void MovePicker::score_evasions_or_checks() { +void MovePicker::score_evasions() { // 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 @@ -261,7 +252,7 @@ void MovePicker::score_evasions_or_checks() { cur->score = pos.midgame_value_of_piece_on(move_to(m)) - pos.type_of_piece_on(move_from(m)) + HistoryMax; else - cur->score = H.move_ordering_score(pos.piece_on(move_from(m)), move_to(m)); + cur->score = H.value(pos.piece_on(move_from(m)), move_to(m)); } }