jirayu_sri/stockfish
1
0
Fork 0
mirror of https://github.com/official-stockfish/Stockfish.git synced 2026-07-22 12:47:08 +00:00
Code Issues Packages Projects Releases Wiki Activity

Reuse Promotion Type Natively

Browse Source 
Piece promotion is generated, and then type_of(promotion) is used to retrieve
the underlying type. The class Move provides .promotion_type() natively,
meaning we can cache the type directly and avoid extracting it indirectly
afterward.  By extracting pt = m.promotion_type(); up front, we don't need to
call type_of(promotion) iteratively inside the assertions or conditions below.

closes https://github.com/official-stockfish/Stockfish/pull/6759

No functional change
This commit is contained in:
FauziAkram
2026-04-26 09:25:45 +02:00
committed by Joost VandeVondele
parent 50e8ff1e23
commit 38d887ac64
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/position.cpp
+4 -4
View File
@@ -996,11 +996,11 @@ void Position::do_move(Move m,
else if (m.type_of() == PROMOTION)
{
Piece promotion = make_piece(us, m.promotion_type());
PieceType promotionType = type_of(promotion);
PieceType pt = m.promotion_type();
Piece promotion = make_piece(us, pt);
assert(relative_rank(us, to) == RANK_8);
assert(type_of(promotion) >= KNIGHT && type_of(promotion) <= QUEEN);
assert(pt >= KNIGHT && pt <= QUEEN);
dp.add_pc = promotion;
dp.add_sq = to;
@@ -1013,7 +1013,7 @@ void Position::do_move(Move m,
^ Zobrist::psq[pc][8 + pieceCount[pc]];
st->nonPawnKey[us] ^= Zobrist::psq[promotion][to];
if (promotionType <= BISHOP)
if (pt <= BISHOP)
st->minorPieceKey ^= Zobrist::psq[promotion][to];
// Update material