From: Fabian Fichter <ianfab@users.noreply.github.com>
Date: Tue, 23 Jan 2018 13:05:48 +0000 (+0100)
Subject: Simplify away redundant SEE pruning condition (#1363)
X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=b61759e907e508d436b7c0b7ff8ab866454f7ca6;ds=sidebyside

Simplify away redundant SEE pruning condition (#1363)

SEE immediately returns true for promotions,
so excluding them before checking SEE is redundant.

STC
LLR: 2.95 (-2.94,2.94) [-3.00,1.00]
Total: 56758 W: 10166 L: 10106 D: 36486
http://tests.stockfishchess.org/tests/view/5a645eaf0ebc590297903833

No functional change.
---

diff --git a/src/search.cpp b/src/search.cpp
index 2fd84207..e0f6cba2 100644
--- a/src/search.cpp
+++ b/src/search.cpp
@@ -1252,7 +1252,6 @@ moves_loop: // When in check search starts from here
 
       // Don't search moves with negative SEE values
       if (  (!InCheck || evasionPrunable)
-          &&  type_of(move) != PROMOTION
           &&  !pos.see_ge(move))
           continue;