mirror of
https://github.com/HChaZZY/Stockfish.git
synced 2025-12-06 10:53:50 +08:00
Replace aligned() function with line_bb() and simplify king piece detection
From https://discord.com/channels/435943710472011776/813919248455827515/1342267241168900228 Saves 6 instructions closes https://github.com/official-stockfish/Stockfish/pull/5909 No functional change
This commit is contained in:
@@ -560,7 +560,7 @@ bool Position::legal(Move m) const {
|
||||
|
||||
// A non-king move is legal if and only if it is not pinned or it
|
||||
// is moving along the ray towards or away from the king.
|
||||
return !(blockers_for_king(us) & from) || aligned(from, to, square<KING>(us));
|
||||
return !(blockers_for_king(us) & from) || line_bb(from, to) & pieces(us, KING);
|
||||
}
|
||||
|
||||
|
||||
@@ -648,7 +648,7 @@ bool Position::gives_check(Move m) const {
|
||||
|
||||
// Is there a discovered check?
|
||||
if (blockers_for_king(~sideToMove) & from)
|
||||
return !aligned(from, to, square<KING>(~sideToMove)) || m.type_of() == CASTLING;
|
||||
return !(line_bb(from, to) & pieces(~sideToMove, KING) || m.type_of() == CASTLING);
|
||||
|
||||
switch (m.type_of())
|
||||
{
|
||||
@@ -656,7 +656,7 @@ bool Position::gives_check(Move m) const {
|
||||
return false;
|
||||
|
||||
case PROMOTION :
|
||||
return attacks_bb(m.promotion_type(), to, pieces() ^ from) & square<KING>(~sideToMove);
|
||||
return attacks_bb(m.promotion_type(), to, pieces() ^ from) & pieces(~sideToMove, KING);
|
||||
|
||||
// En passant capture with check? We have already handled the case of direct
|
||||
// checks and ordinary discovered check, so the only case we need to handle
|
||||
|
||||
Reference in New Issue
Block a user