A move needs 17 bits not 19
authorMarco Costalba <mcostalba@gmail.com>
Sun, 29 Mar 2009 16:22:10 +0000 (17:22 +0100)
committerMarco Costalba <mcostalba@gmail.com>
Sun, 29 Mar 2009 16:23:41 +0000 (17:23 +0100)
Fix a bug in the way a move is stored and read in a TT entry.
We use a mask of 19 bits insteaad of 17 so that the last
two bits in the TT entry end up to be random data.

This bug will bite us when we will use these two until now
unused bits.

Signed-off-by: Marco Costalba <mcostalba@gmail.com>
src/tt.cpp
src/tt.h

index c88aef4..5c842db 100644 (file)
@@ -206,5 +206,5 @@ TTEntry::TTEntry() {
 
 TTEntry::TTEntry(Key k, Value v, ValueType t, Depth d, Move m,
                  int generation) :
-  key_ (k), data((m & 0x7FFFF) | (t << 20) | (generation << 23)),
+  key_ (k), data((m & 0x1FFFF) | (t << 20) | (generation << 23)),
   value_(int16_t(v)), depth_(int16_t(d)) {}
index 02913ef..31fc1fb 100644 (file)
--- a/src/tt.h
+++ b/src/tt.h
@@ -43,7 +43,7 @@ public:
   TTEntry(Key k, Value v, ValueType t, Depth d, Move m, int generation);
   Key key() const { return key_; }
   Depth depth() const { return Depth(depth_); }
-  Move move() const { return Move(data & 0x7FFFF); }
+  Move move() const { return Move(data & 0x1FFFF); }
   Value value() const { return Value(value_); }
   ValueType type() const { return ValueType((data >> 20) & 7); }
   int generation() const { return (data >> 23); }