From: bmc4 Date: Mon, 15 Mar 2021 22:52:45 +0000 (-0300) Subject: Simplify move generation (1/2) X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=4b509559fbabe8a41cb8387c71d07bb1c7b78d6f Simplify move generation (1/2) STC: LLR: 2.95 (-2.94,2.94) {-1.25,0.25} Total: 29792 W: 2611 L: 2545 D: 24636 Ptnml(0-2): 94, 1982, 10659, 2086, 75 https://tests.stockfishchess.org/tests/view/604fe5b62433018de7a38ba8 LTC: LLR: 2.92 (-2.94,2.94) {-0.75,0.25} Total: 22040 W: 826 L: 777 D: 20437 Ptnml(0-2): 8, 646, 9664, 693, 9 https://tests.stockfishchess.org/tests/view/604fec892433018de7a38bac Closes https://github.com/official-stockfish/Stockfish/pull/3399 No functional change --- diff --git a/src/movegen.cpp b/src/movegen.cpp index c5d76afa..d8c4370a 100644 --- a/src/movegen.cpp +++ b/src/movegen.cpp @@ -182,17 +182,12 @@ namespace { Bitboard bb = piecesToMove & pos.pieces(Pt); - if (!bb) - return moveList; - - [[maybe_unused]] const Bitboard checkSquares = pos.check_squares(Pt); - while (bb) { Square from = pop_lsb(&bb); Bitboard b = attacks_bb(from, pos.pieces()) & target; if constexpr (Checks) - b &= checkSquares; + b &= pos.check_squares(Pt); while (b) *moveList++ = make_move(from, pop_lsb(&b));