From: Marco Costalba <mcostalba@gmail.com>
Date: Sat, 1 Dec 2012 14:12:18 +0000 (+0100)
Subject: Don't use TT just to save a node evaluation
X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=2a585b63b80f5c3312fd0f6f250ddb337e87481b;ds=sidebyside

Don't use TT just to save a node evaluation

In search(), after we evalute the position, in case there
isn't any TT entry we create one with just the evaluation
score.

This patches removes that code. The reason becuase the patch
deserves a single commit it is becuase introduces a (very small)
functional change due to the fact that the total number of
TT stores is less now and this slightly alters the TT hits
of our benchmark.

bench: 4983262
---

diff --git a/src/search.cpp b/src/search.cpp
index 79046ad2..1109bf8e 100644
--- a/src/search.cpp
+++ b/src/search.cpp
@@ -585,10 +585,6 @@ namespace {
                 || ((tte->type() & BOUND_UPPER) && ttValue < eval))
                 eval = ttValue;
         }
-
-        if (!tte)
-            TT.store(posKey, VALUE_NONE, BOUND_NONE, DEPTH_NONE, MOVE_NONE,
-                     ss->staticEval, ss->evalMargin);
     }
 
     // Update gain for the parent non-capture move given the static position