From d2274e609c28e46a24deee55b05684197d369976 Mon Sep 17 00:00:00 2001 From: protonspring Date: Mon, 19 Nov 2018 10:37:07 +0100 Subject: [PATCH] Remove BlockedStorm array Apparently, only RANK_3 is relevant. This removes a look-up and the BlockedStorm array, but adds another conditional. STC LLR: 2.96 (-2.94,2.94) [-3.00,1.00] Total: 84340 W: 18054 L: 18054 D: 48232 http://tests.stockfishchess.org/tests/view/5bea10f40ebc595e0ae3457b LTC LLR: 2.95 (-2.94,2.94) [-3.00,1.00] Total: 31874 W: 5135 L: 5032 D: 21707 http://tests.stockfishchess.org/tests/view/5beadb6a0ebc595e0ae35542 Closes https://github.com/official-stockfish/Stockfish/pull/1814 Bench: 3799443 --- src/pawns.cpp | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/src/pawns.cpp b/src/pawns.cpp index 98e13f83..ea0dffc7 100644 --- a/src/pawns.cpp +++ b/src/pawns.cpp @@ -58,10 +58,6 @@ namespace { { V(-10), V(-14), V( 90), V(15), V( 2), V( -7), V(-16) } }; - // Danger of blocked enemy pawns storming our king, by rank - constexpr Value BlockedStorm[RANK_NB] = - { V(0), V(0), V(66), V(6), V(5), V(1), V(15) }; - #undef S #undef V @@ -225,7 +221,7 @@ Value Entry::evaluate_shelter(const Position& pos, Square ksq) { int d = std::min(f, ~f); safety += ShelterStrength[d][ourRank]; - safety -= (ourRank && (ourRank == theirRank - 1)) ? BlockedStorm[theirRank] + safety -= (ourRank && (ourRank == theirRank - 1)) ? 66 * (theirRank == RANK_3) : UnblockedStorm[d][theirRank]; } -- 2.39.2