Simplify away redundant SEE pruning condition (#1363)
authorFabian Fichter <ianfab@users.noreply.github.com>
Tue, 23 Jan 2018 13:05:48 +0000 (14:05 +0100)
committerMarco Costalba <mcostalba@users.noreply.github.com>
Tue, 23 Jan 2018 13:05:48 +0000 (14:05 +0100)
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.

src/search.cpp

index 2fd8420..e0f6cba 100644 (file)
@@ -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;