mirror of
https://github.com/HChaZZY/Stockfish.git
synced 2025-12-17 07:36:23 +08:00
Fixed a bug in PV extraction from the transposition table: The
previous used move_is_legal to verify that the move from the TT was legal, and the old version of move_is_legal only works when the side to move is not in check. Fixed this by adding a separate, slower version of move_is_legal which works even when the side to move is in check.
This commit is contained in:
@@ -567,6 +567,29 @@ bool move_is_legal(const Position& pos, const Move m, Bitboard pinned) {
|
||||
}
|
||||
|
||||
|
||||
/// Another version of move_is_legal(), which takes only a position and a move
|
||||
/// as input. This function does not require that the side to move is not in
|
||||
/// check. It is not optimized for speed, and is only used for verifying move
|
||||
/// legality when building a PV from the transposition table.
|
||||
|
||||
bool move_is_legal(const Position& pos, const Move m) {
|
||||
|
||||
Bitboard pinned = pos.pinned_pieces(pos.side_to_move());
|
||||
if (!pos.is_check())
|
||||
return move_is_legal(pos, m, pinned);
|
||||
else
|
||||
{
|
||||
Position p(pos);
|
||||
MoveStack moves[64];
|
||||
int n = generate_evasions(p, moves, pinned);
|
||||
for (int i = 0; i < n; i++)
|
||||
if (moves[i].move == m)
|
||||
return true;
|
||||
return false;
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
namespace {
|
||||
|
||||
template<PieceType Piece>
|
||||
|
||||
Reference in New Issue
Block a user