const History& h, SearchStack* ss) : pos(p), H(h) {
int searchTT = ttm;
ttMoves[0].move = ttm;
- finished = false;
lastBadCapture = badCaptures;
pinned = p.pinned_pieces(pos.side_to_move());
case PH_EVASIONS:
assert(pos.is_check());
lastMove = generate_evasions(pos, moves);
- score_evasions();
+ score_evasions_or_checks();
return;
case PH_QCAPTURES:
return;
case PH_QCHECKS:
- // Perhaps we should order moves move here? FIXME
lastMove = generate_non_capture_checks(pos, moves);
+ score_evasions_or_checks();
return;
case PH_STOP:
- lastMove = curMove + 1; // hack to be friendly for get_next_move()
+ lastMove = curMove + 1; // Avoids another go_next_phase() call
return;
default:
}
}
-void MovePicker::score_evasions() {
+void MovePicker::score_evasions_or_checks() {
// 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
Move m;
int seeScore;
+ // Skip if we don't have at least two moves to order
+ if (lastMove < moves + 2)
+ return;
+
for (MoveStack* cur = moves; cur != lastMove; cur++)
{
m = cur->move;
/// 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.
+/// Note that this function is not thread safe so should be lock protected by
+/// caller when accessed through a shared MovePicker object.
Move MovePicker::get_next_move() {
}
}
-/// 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.
-
-Move MovePicker::get_next_move(Lock &lock) {
-
- lock_grab(&lock);
- if (finished)
- {
- lock_release(&lock);
- return MOVE_NONE;
- }
- Move m = get_next_move();
- if (m == MOVE_NONE)
- finished = true;
-
- lock_release(&lock);
- return m;
-}