From: Marco Costalba <mcostalba@gmail.com>
Date: Tue, 27 Nov 2012 19:05:03 +0000 (+0100)
Subject: Don't double check for move legality
X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=4502917665d50394ead903d0b82c6e7b4777b99d;hp=5af8179647b9e80353cb8f532990cae572ba6412

Don't double check for move legality

In case of a RootNode or a SpNode move has
been already checked for legality so we can
skip a redundant check.

Spotted by Frank Genot.

No functional change.
---

diff --git a/src/search.cpp b/src/search.cpp
index 3116e80e..da20f15e 100644
--- a/src/search.cpp
+++ b/src/search.cpp
@@ -893,7 +893,7 @@ split_point_start: // At split points actual search starts from here
       }
 
       // Check for legality only before to do the move
-      if (!pos.pl_move_is_legal(move, ci.pinned))
+      if (!RootNode && !SpNode && !pos.pl_move_is_legal(move, ci.pinned))
       {
           moveCount--;
           continue;