]> git.sesse.net Git - stockfish/commitdiff
Merge remote-tracking branch 'upstream/master' into HEAD
authorSteinar H. Gunderson <sgunderson@bigfoot.com>
Sun, 14 Mar 2021 15:58:42 +0000 (16:58 +0100)
committerSteinar H. Gunderson <sgunderson@bigfoot.com>
Sun, 14 Mar 2021 15:58:42 +0000 (16:58 +0100)
1  2 
src/Makefile
src/main.cpp
src/misc.cpp
src/position.cpp
src/syzygy/tbprobe.cpp
src/ucioption.cpp

diff --combined src/Makefile
index e3466ea1bb8791979a6430d27cd35f1b15c50315,eb32758f98de016e6fa795ade98f47103646558d..d7e09c8c682b510872f613c10118b12118b2cea9
@@@ -1,7 -1,5 +1,5 @@@
  # Stockfish, a UCI chess playing engine derived from Glaurung 2.1
- # Copyright (C) 2004-2008 Tord Romstad (Glaurung author)
- # Copyright (C) 2008-2015 Marco Costalba, Joona Kiiski, Tord Romstad
- # Copyright (C) 2015-2019 Marco Costalba, Joona Kiiski, Gary Linscott, Tord Romstad
+ # Copyright (C) 2004-2021 The Stockfish developers (see AUTHORS file)
  #
  # Stockfish is free software: you can redistribute it and/or modify
  # it under the terms of the GNU General Public License as published by
@@@ -39,12 -37,9 +37,12 @@@ PGOBENCH = ./$(EXE) benc
  SRCS = benchmark.cpp bitbase.cpp bitboard.cpp endgame.cpp evaluate.cpp main.cpp \
        material.cpp misc.cpp movegen.cpp movepick.cpp pawns.cpp position.cpp psqt.cpp \
        search.cpp thread.cpp timeman.cpp tt.cpp uci.cpp ucioption.cpp tune.cpp syzygy/tbprobe.cpp \
 -      nnue/evaluate_nnue.cpp nnue/features/half_kp.cpp
 +      nnue/evaluate_nnue.cpp nnue/features/half_kp.cpp \
 +      hashprobe.grpc.pb.cc hashprobe.pb.cc
 +CLISRCS = client.cpp hashprobe.grpc.pb.cc hashprobe.pb.cc uci.cpp
  
  OBJS = $(notdir $(SRCS:.cpp=.o))
 +CLIOBJS = $(notdir $(CLISRCS:.cpp=.o))
  
  VPATH = syzygy:nnue:nnue/features
  
@@@ -310,7 -305,7 +308,7 @@@ endi
  ifeq ($(COMP),gcc)
        comp=gcc
        CXX=g++
 -      CXXFLAGS += -pedantic -Wextra -Wshadow
 +      CXXFLAGS += -pedantic -Wextra
  
        ifeq ($(arch),$(filter $(arch),armv7 armv8))
                ifeq ($(OS),Android)
@@@ -369,9 -364,11 +367,11 @@@ ifeq ($(COMP),clang
  
        ifneq ($(KERNEL),Darwin)
        ifneq ($(KERNEL),OpenBSD)
+       ifneq ($(KERNEL),FreeBSD)
                LDFLAGS += -latomic
        endif
        endif
+       endif
  
        ifeq ($(arch),$(filter $(arch),armv7 armv8))
                ifeq ($(OS),Android)
@@@ -469,7 -466,7 +469,7 @@@ endi
  ### 3.3 Optimization
  ifeq ($(optimize),yes)
  
 -      CXXFLAGS += -O3
 +      CXXFLAGS += -O3 -g
  
        ifeq ($(comp),gcc)
                ifeq ($(OS), Android)
                        CXXFLAGS += -mdynamic-no-pic
                endif
        endif
+       ifeq ($(comp),clang)
+               CXXFLAGS += -fexperimental-new-pass-manager
+       endif
  endif
  
  ### 3.4 Bits
@@@ -591,7 -592,7 +595,7 @@@ endi
  ifeq ($(optimize),yes)
  ifeq ($(debug), no)
        ifeq ($(comp),clang)
-               CXXFLAGS += -flto=thin
+               CXXFLAGS += -flto
                ifneq ($(findstring MINGW,$(KERNEL)),)
                        CXXFLAGS += -fuse-ld=lld
                else ifneq ($(findstring MSYS,$(KERNEL)),)
                        LDFLAGS += -save-temps
                endif
        else
-               CXXFLAGS += -flto=thin
+               CXXFLAGS += -flto
                LDFLAGS += $(CXXFLAGS)
        endif
  
@@@ -785,7 -786,7 +789,7 @@@ default
  ### Section 5. Private Targets
  ### ==========================================================================
  
 -all: $(EXE) .depend
 +all: $(EXE) client .depend
  
  config-sanity: net
        @echo ""
@@@ -881,32 -882,6 +885,32 @@@ icc-profile-use
        EXTRACXXFLAGS='-prof_use -prof_dir ./profdir' \
        all
  
 +### GRPC
 +
 +PROTOS_PATH = .
 +PROTOC = protoc
 +GRPC_CPP_PLUGIN = grpc_cpp_plugin
 +GRPC_CPP_PLUGIN_PATH ?= `which $(GRPC_CPP_PLUGIN)`
 +
 +%.grpc.pb.h %.grpc.pb.cc: %.proto
 +      $(PROTOC) -I $(PROTOS_PATH) --grpc_out=. --plugin=protoc-gen-grpc=$(GRPC_CPP_PLUGIN_PATH) $<
 +
 +# oh my
 +%.cpp: %.cc
 +      cp $< $@
 +
 +%.pb.h %.pb.cc: %.proto
 +      $(PROTOC) -I $(PROTOS_PATH) --cpp_out=. $<
 +
 +#LDFLAGS += -Wl,-Bstatic -Wl,-\( -lprotobuf -lgrpc++_unsecure -lgrpc_unsecure -lgrpc -lz -Wl,-\) -Wl,-Bdynamic -ldl
 +LDFLAGS += /usr/lib/x86_64-linux-gnu/libprotobuf.a /usr/lib/x86_64-linux-gnu/libgrpc++_unsecure.a /usr/lib/x86_64-linux-gnu/libgrpc_unsecure.a /usr/lib/x86_64-linux-gnu/libgrpc.a /usr/lib/x86_64-linux-gnu/libcares.a -ldl -lz
 +#LDFLAGS += /usr/lib/x86_64-linux-gnu/libprotobuf.a /usr/lib/libgrpc++_unsecure.a /usr/lib/libgrpc_unsecure.a /usr/lib/libgrpc.a /usr/lib/x86_64-linux-gnu/libcares.a -ldl -lz
 +
 +client: $(CLIOBJS)
 +      $(CXX) -o $@ $(CLIOBJS) $(LDFLAGS)
 +
 +# Other stuff
 +
  .depend:
        -@$(CXX) $(DEPENDFLAGS) -MM $(SRCS) > $@ 2> /dev/null
  
diff --combined src/main.cpp
index e8548ac1a451bdb4c4204ec5a4e0df315397c4ed,62e0ed525d709c0cb3e6a21d44571684e702467a..a2c741e53e2e4bd0f9bbfa458d6ceb69932a909f
@@@ -1,6 -1,6 +1,6 @@@
  /*
    Stockfish, a UCI chess playing engine derived from Glaurung 2.1
-   Copyright (C) 2004-2020 The Stockfish developers (see AUTHORS file)
+   Copyright (C) 2004-2021 The Stockfish developers (see AUTHORS file)
  
    Stockfish is free software: you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
    along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
  
 +#include <deque>
  #include <iostream>
 +#include <stack>
 +#include <thread>
  
  #include "bitboard.h"
  #include "endgame.h"
  #include "position.h"
+ #include "psqt.h"
  #include "search.h"
+ #include "syzygy/tbprobe.h"
  #include "thread.h"
  #include "tt.h"
  #include "uci.h"
- #include "syzygy/tbprobe.h"
  
 +#include <grpc/grpc.h>
 +#include <grpc++/server.h>
 +#include <grpc++/server_builder.h>
 +#include "hashprobe.h"
 +#include "hashprobe.grpc.pb.h"
 +#include "tt.h"
 +
 +using grpc::Server;
 +using grpc::ServerBuilder;
 +using grpc::ServerContext;
 +using grpc::Status;
 +using grpc::StatusCode;
 +using namespace hashprobe;
 +
 +Status HashProbeImpl::Probe(ServerContext* context,
 +                            const HashProbeRequest* request,
 +                          HashProbeResponse *response) {
 +      Position pos;
 +      StateInfo st;
 +      pos.set(request->fen(), /*isChess960=*/false, &st, Threads.main());
 +      if (!pos.pos_is_ok()) {
 +              return Status(StatusCode::INVALID_ARGUMENT, "Invalid FEN");
 +      }
 +
 +      bool invert = (pos.side_to_move() == BLACK);
 +      StateListPtr setup_states = StateListPtr(new std::deque<StateInfo>(1));
 +
 +      ProbeMove(&pos, setup_states.get(), invert, response->mutable_root());
 +
 +      MoveList<LEGAL> moves(pos);
 +      for (const ExtMove* em = moves.begin(); em != moves.end(); ++em) {
 +              HashProbeLine *line = response->add_line();
 +              FillMove(&pos, em->move, line->mutable_move());
 +              setup_states->push_back(StateInfo());
 +              pos.do_move(em->move, setup_states->back());
 +              ProbeMove(&pos, setup_states.get(), !invert, line);
 +              pos.undo_move(em->move);
 +      }
 +
 +      return Status::OK;
 +}
 +
 +void HashProbeImpl::FillMove(Position *pos, Move move, HashProbeMove* decoded) {
 +      if (!is_ok(move)) return;
 +
 +      Square from = from_sq(move);
 +      Square to = to_sq(move);
 +
 +      if (type_of(move) == CASTLING) {
 +              to = make_square(to > from ? FILE_G : FILE_C, rank_of(from));
 +      }
 +
 +      Piece moved_piece = pos->moved_piece(move);
 +      std::string pretty;
 +      if (type_of(move) == CASTLING) {
 +              if (to > from) {
 +                      pretty = "O-O";
 +              } else {
 +                      pretty = "O-O-O";
 +              }
 +      } else if (type_of(moved_piece) == PAWN) {
 +              if (type_of(move) == ENPASSANT || pos->piece_on(to) != NO_PIECE) {
 +                      // Capture.
 +                      pretty = char('a' + file_of(from));
 +                      pretty += "x";
 +              }
 +              pretty += UCI::square(to);
 +              if (type_of(move) == PROMOTION) {
 +                      pretty += "=";
 +                      pretty += " PNBRQK"[promotion_type(move)];
 +              }
 +      } else {
 +              pretty = " PNBRQK"[type_of(moved_piece)];
 +              Bitboard attackers = pos->attackers_to(to) & pos->pieces(color_of(moved_piece), type_of(moved_piece));
 +              if (more_than_one(attackers)) {
 +                      // Remove all illegal moves to disambiguate.
 +                      Bitboard att_copy = attackers;
 +                      while (att_copy) {
 +                              Square s = pop_lsb(&att_copy);
 +                              Move m = make_move(s, to);
 +                              if (!pos->pseudo_legal(m) || !pos->legal(m)) {
 +                                      attackers &= ~SquareBB[s];
 +                              }
 +                      }
 +              }
 +              if (more_than_one(attackers)) {
 +                      // Disambiguate by file if possible.
 +                      Bitboard attackers_this_file = attackers & file_bb(file_of(from));
 +                      if (attackers != attackers_this_file) {
 +                              pretty += char('a' + file_of(from));
 +                              attackers = attackers_this_file;
 +                      }
 +                      if (more_than_one(attackers)) {
 +                              // Still ambiguous, so need to disambiguate by rank.
 +                              pretty += char('1' + rank_of(from));
 +                      }
 +              }
 +
 +              if (type_of(move) == ENPASSANT || pos->piece_on(to) != NO_PIECE) {
 +                      pretty += "x";
 +              }
 +
 +              pretty += UCI::square(to);
 +      }
 +
 +      if (pos->gives_check(move)) {
 +              // Check if mate.
 +              StateInfo si;
 +              pos->do_move(move, si, true);
 +              if (MoveList<LEGAL>(*pos).size() > 0) {
 +                      pretty += "+";
 +              } else {
 +                      pretty += "#";
 +              }
 +              pos->undo_move(move);
 +      }
 +
 +      decoded->set_pretty(pretty);
 +}
 +
 +void HashProbeImpl::ProbeMove(Position* pos, std::deque<StateInfo>* setup_states, bool invert, HashProbeLine* response) {
 +      bool found;
 +      TTEntry *entry = TT.probe(pos->key(), found);
 +      response->set_found(found);
 +      if (found) {
 +              Value value = entry->value();
 +              Value eval = entry->eval();
 +              Bound bound = entry->bound();
 +
 +              if (invert) {
 +                      value = -value;
 +                      eval = -eval;
 +                      if (bound == BOUND_UPPER) {
 +                              bound = BOUND_LOWER;
 +                      } else if (bound == BOUND_LOWER) {
 +                              bound = BOUND_UPPER;
 +                      }
 +              }
 +
 +              response->set_depth(entry->depth());
 +              FillValue(eval, response->mutable_eval());
 +              if (entry->depth() > DEPTH_NONE) {
 +                      FillValue(value, response->mutable_value());
 +              }
 +              response->set_bound(HashProbeLine::ValueBound(bound));
 +
 +              // Follow the PV until we hit an illegal move.
 +              std::stack<Move> pv;
 +              std::set<Key> seen;
 +              while (found && is_ok(entry->move()) &&
 +                     pos->pseudo_legal(entry->move()) &&
 +                     pos->legal(entry->move())) {
 +                      FillMove(pos, entry->move(), response->add_pv());
 +                      if (seen.count(pos->key())) break;
 +                      pv.push(entry->move());
 +                      seen.insert(pos->key());
 +                      setup_states->push_back(StateInfo());
 +                      pos->do_move(entry->move(), setup_states->back());
 +                      entry = TT.probe(pos->key(), found);
 +              }
 +
 +              // Unroll the PV back again, so the Position object remains unchanged.
 +              while (!pv.empty()) {
 +                      pos->undo_move(pv.top());
 +                      pv.pop();
 +              }
 +      }
 +}
 +
 +void HashProbeImpl::FillValue(Value value, HashProbeScore* score) {
 +      if (abs(value) < VALUE_MATE - MAX_PLY) {
 +              score->set_score_type(HashProbeScore::SCORE_CP);
 +              score->set_score_cp(value * 100 / PawnValueEg);
 +      } else {
 +              score->set_score_type(HashProbeScore::SCORE_MATE);
 +              score->set_score_mate((value > 0 ? VALUE_MATE - value + 1 : -VALUE_MATE - value) / 2);
 +      }
 +}
 +
 +HashProbeThread::HashProbeThread(const std::string &server_address) {
 +      builder.AddListeningPort(server_address, grpc::InsecureServerCredentials());
 +      builder.RegisterService(&service);
 +      server = std::move(builder.BuildAndStart());
 +      std::cout << "Server listening on " << server_address << std::endl;
 +      std::thread([this]{ server->Wait(); }).detach();
 +}
 +
 +void HashProbeThread::Shutdown() {
 +      server->Shutdown();
 +}
 +
 +namespace PSQT {
 +  void init();
 +}
 +
+ using namespace Stockfish;
  int main(int argc, char* argv[]) {
  
    std::cout << engine_info() << std::endl;
diff --combined src/misc.cpp
index 832a9ac18f93465baca9eb9c54de107453c71916,7600fc114c2f6a9d411170935556526d4bf22a43..81602d71101690f09418990da1c94815e51e3d98
@@@ -1,6 -1,6 +1,6 @@@
  /*
    Stockfish, a UCI chess playing engine derived from Glaurung 2.1
-   Copyright (C) 2004-2020 The Stockfish developers (see AUTHORS file)
+   Copyright (C) 2004-2021 The Stockfish developers (see AUTHORS file)
  
    Stockfish is free software: you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
@@@ -61,6 -61,8 +61,8 @@@ typedef bool(*fun3_t)(HANDLE, CONST GRO
  
  using namespace std;
  
+ namespace Stockfish {
  namespace {
  
  /// Version number. If Version is left empty, then compile date in the format
@@@ -138,7 -140,7 +140,7 @@@ public
  /// the program was compiled) or "Stockfish <Version>", depending on whether
  /// Version is empty.
  
const string engine_info(bool to_uci) {
+ string engine_info(bool to_uci) {
  
    const string months("Jan Feb Mar Apr May Jun Jul Aug Sep Oct Nov Dec");
    string month, day, year;
    {
        date >> month >> day >> year;
        ss << setw(2) << day << setw(2) << (1 + months.find(month) / 4) << year.substr(2);
 +      ss << "-asn";
    }
  
    ss << (to_uci  ? "\nid author ": " by ")
  
  /// compiler_info() returns a string trying to describe the compiler we use
  
const std::string compiler_info() {
+ std::string compiler_info() {
  
    #define stringify2(x) #x
    #define stringify(x) stringify2(x)
@@@ -361,7 -362,7 +363,7 @@@ void std_aligned_free(void* ptr) 
  /// aligned_large_pages_alloc() will return suitably aligned memory, if possible using large pages.
  
  #if defined(_WIN32)
+ #if defined(_WIN64)
  static void* aligned_large_pages_alloc_win(size_t allocSize) {
  
    HANDLE hProcessToken { };
  
    return mem;
  }
+ #endif
  
  void* aligned_large_pages_alloc(size_t allocSize) {
  
+ #if defined(_WIN64)
    // Try to allocate large pages
    void* mem = aligned_large_pages_alloc_win(allocSize);
  
    // Fall back to regular, page aligned, allocation if necessary
    if (!mem)
        mem = VirtualAlloc(NULL, allocSize, MEM_RESERVE | MEM_COMMIT, PAGE_READWRITE);
+ #else
+   void* mem = VirtualAlloc(NULL, allocSize, MEM_RESERVE | MEM_COMMIT, PAGE_READWRITE);
+ #endif
  
    return mem;
  }
@@@ -627,3 -633,5 +634,5 @@@ void init(int argc, char* argv[]) 
  
  
  } // namespace CommandLine
+ } // namespace Stockfish
diff --combined src/position.cpp
index 13010c1a43f49571193d745a144ad92a4867bb9d,17b165b9afb6f9a4115afc03d0762e98076993b4..a2ee64f81cc603e6e4a03dbbfbbc00afeb863aa6
@@@ -1,6 -1,6 +1,6 @@@
  /*
    Stockfish, a UCI chess playing engine derived from Glaurung 2.1
-   Copyright (C) 2004-2020 The Stockfish developers (see AUTHORS file)
+   Copyright (C) 2004-2021 The Stockfish developers (see AUTHORS file)
  
    Stockfish is free software: you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
@@@ -34,6 -34,8 +34,8 @@@
  
  using std::string;
  
+ namespace Stockfish {
  namespace Zobrist {
  
    Key psq[PIECE_NB][SQUARE_NB];
@@@ -249,6 -251,8 +251,8 @@@ Position& Position::set(const string& f
        set_castling_right(c, rsq);
    }
  
+   set_state(st);
    // 4. En passant square.
    // Ignore if square is invalid or not on side to move relative rank 6.
    bool enpassant = false;
        // a) side to move have a pawn threatening epSquare
        // b) there is an enemy pawn in front of epSquare
        // c) there is no piece on epSquare or behind epSquare
+       // d) enemy pawn didn't block a check of its own color by moving forward
        enpassant = pawn_attacks_bb(~sideToMove, st->epSquare) & pieces(sideToMove, PAWN)
                 && (pieces(~sideToMove, PAWN) & (st->epSquare + pawn_push(~sideToMove)))
-                && !(pieces() & (st->epSquare | (st->epSquare + pawn_push(sideToMove))));
+                && !(pieces() & (st->epSquare | (st->epSquare + pawn_push(sideToMove))))
+                && (   file_of(square<KING>(sideToMove)) == file_of(st->epSquare)
+                    || !(blockers_for_king(sideToMove) & (st->epSquare + pawn_push(~sideToMove))));
    }
  
-   if (!enpassant)
+   // It's necessary for st->previous to be intialized in this way because legality check relies on its existence
+   if (enpassant) {
+       st->previous = new StateInfo();
+       remove_piece(st->epSquare - pawn_push(sideToMove));
+       st->previous->checkersBB = attackers_to(square<KING>(~sideToMove)) & pieces(sideToMove);
+       st->previous->blockersForKing[WHITE] = slider_blockers(pieces(BLACK), square<KING>(WHITE), st->previous->pinners[BLACK]);
+       st->previous->blockersForKing[BLACK] = slider_blockers(pieces(WHITE), square<KING>(BLACK), st->previous->pinners[WHITE]);
+       put_piece(make_piece(~sideToMove, PAWN), st->epSquare - pawn_push(sideToMove));
+   }
+   else
        st->epSquare = SQ_NONE;
  
    // 5-6. Halfmove clock and fullmove number
  
    chess960 = isChess960;
    thisThread = th;
-   set_state(st);
    st->accumulator.state[WHITE] = Eval::NNUE::INIT;
    st->accumulator.state[BLACK] = Eval::NNUE::INIT;
  
 -  assert(pos_is_ok());
 -
    return *this;
  }
  
@@@ -393,7 -410,7 +408,7 @@@ Position& Position::set(const string& c
  /// Position::fen() returns a FEN representation of the position. In case of
  /// Chess960 the Shredder-FEN notation is used. This is mainly a debugging function.
  
const string Position::fen() const {
+ string Position::fen() const {
  
    int emptyCnt;
    std::ostringstream ss;
@@@ -500,23 -517,11 +515,11 @@@ bool Position::legal(Move m) const 
    assert(color_of(moved_piece(m)) == us);
    assert(piece_on(square<KING>(us)) == make_piece(us, KING));
  
-   // En passant captures are a tricky special case. Because they are rather
-   // uncommon, we do it simply by testing whether the king is attacked after
-   // the move is made.
-   if (type_of(m) == ENPASSANT)
-   {
-       Square ksq = square<KING>(us);
-       Square capsq = to - pawn_push(us);
-       Bitboard occupied = (pieces() ^ from ^ capsq) | to;
-       assert(to == ep_square());
-       assert(moved_piece(m) == make_piece(us, PAWN));
-       assert(piece_on(capsq) == make_piece(~us, PAWN));
-       assert(piece_on(to) == NO_PIECE);
-       return   !(attacks_bb<  ROOK>(ksq, occupied) & pieces(~us, QUEEN, ROOK))
-             && !(attacks_bb<BISHOP>(ksq, occupied) & pieces(~us, QUEEN, BISHOP));
-   }
+   // st->previous->blockersForKing consider capsq as empty.
+   // If pinned, it has to move along the king ray.
+   if (type_of(m) == EN_PASSANT)
+       return   !(st->previous->blockersForKing[sideToMove] & from)
+             || aligned(from, to, square<KING>(us));
  
    // Castling moves generation does not check if the castling path is clear of
    // enemy attacks, it is delayed at a later time: now!
            if (attackers_to(s) & pieces(~us))
                return false;
  
-       // In case of Chess960, verify that when moving the castling rook we do
-       // not discover some hidden checker.
+       // In case of Chess960, verify if the Rook blocks some checks
        // For instance an enemy queen in SQ_A1 when castling rook is in SQ_B1.
-       return   !chess960
-             || !(attacks_bb<ROOK>(to, pieces() ^ to_sq(m)) & pieces(~us, ROOK, QUEEN));
+       return !chess960 || !(blockers_for_king(us) & to_sq(m));
    }
  
    // If the moving piece is a king, check whether the destination square is
  
    // A non-king move is legal if and only if it is not pinned or it
    // is moving along the ray towards or away from the king.
-   return   !(blockers_for_king(us) & from)
-         ||  aligned(from, to, square<KING>(us));
+   return !(blockers_for_king(us) & from)
+       || aligned(from, to, square<KING>(us));
  }
  
  
@@@ -562,8 -565,10 +563,10 @@@ bool Position::pseudo_legal(const Move 
    Piece pc = moved_piece(m);
  
    // Use a slower but simpler function for uncommon cases
+   // yet we skip the legality check of MoveList<LEGAL>().
    if (type_of(m) != NORMAL)
-       return MoveList<LEGAL>(*this).contains(m);
+       return checkers() ? MoveList<    EVASIONS>(*this).contains(m)
+                         : MoveList<NON_EVASIONS>(*this).contains(m);
  
    // Is not a promotion, so promotion piece must be empty
    if (promotion_type(m) - KNIGHT != NO_PIECE_TYPE)
@@@ -649,31 -654,24 +652,24 @@@ bool Position::gives_check(Move m) cons
    case PROMOTION:
        return attacks_bb(promotion_type(m), to, pieces() ^ from) & square<KING>(~sideToMove);
  
-   // En passant capture with check? We have already handled the case
-   // of direct checks and ordinary discovered check, so the only case we
-   // need to handle is the unusual case of a discovered check through
-   // the captured pawn.
-   case ENPASSANT:
-   {
-       Square capsq = make_square(file_of(to), rank_of(from));
-       Bitboard b = (pieces() ^ from ^ capsq) | to;
+   // The double-pushed pawn blocked a check? En Passant will remove the blocker.
+   // The only discovery check that wasn't handle is through capsq and fromsq
+   // So the King must be in the same rank as fromsq to consider this possibility.
+   // st->previous->blockersForKing consider capsq as empty.
+   case EN_PASSANT:
+       return st->previous->checkersBB
+           || (   rank_of(square<KING>(~sideToMove)) == rank_of(from)
+               && st->previous->blockersForKing[~sideToMove] & from);
  
-       return  (attacks_bb<  ROOK>(square<KING>(~sideToMove), b) & pieces(sideToMove, QUEEN, ROOK))
-             | (attacks_bb<BISHOP>(square<KING>(~sideToMove), b) & pieces(sideToMove, QUEEN, BISHOP));
-   }
-   case CASTLING:
+   default: //CASTLING
    {
-       Square kfrom = from;
-       Square rfrom = to; // Castling is encoded as 'king captures the rook'
-       Square kto = relative_square(sideToMove, rfrom > kfrom ? SQ_G1 : SQ_C1);
-       Square rto = relative_square(sideToMove, rfrom > kfrom ? SQ_F1 : SQ_D1);
+       // Castling is encoded as 'king captures the rook'
+       Square ksq = square<KING>(~sideToMove);
+       Square rto = relative_square(sideToMove, to > from ? SQ_F1 : SQ_D1);
  
-       return   (attacks_bb<ROOK>(rto) & square<KING>(~sideToMove))
-             && (attacks_bb<ROOK>(rto, (pieces() ^ kfrom ^ rfrom) | rto | kto) & square<KING>(~sideToMove));
+       return   (attacks_bb<ROOK>(rto) & ksq)
+             && (attacks_bb<ROOK>(rto, pieces() ^ from ^ to) & ksq);
    }
-   default:
-       assert(false);
-       return false;
    }
  }
  
@@@ -714,7 -712,7 +710,7 @@@ void Position::do_move(Move m, StateInf
    Square from = from_sq(m);
    Square to = to_sq(m);
    Piece pc = piece_on(from);
-   Piece captured = type_of(m) == ENPASSANT ? make_piece(them, PAWN) : piece_on(to);
+   Piece captured = type_of(m) == EN_PASSANT ? make_piece(them, PAWN) : piece_on(to);
  
    assert(color_of(pc) == us);
    assert(captured == NO_PIECE || color_of(captured) == (type_of(m) != CASTLING ? them : us));
        // update non-pawn material.
        if (type_of(captured) == PAWN)
        {
-           if (type_of(m) == ENPASSANT)
+           if (type_of(m) == EN_PASSANT)
            {
                capsq -= pawn_push(us);
  
        // Update board and piece lists
        remove_piece(capsq);
  
-       if (type_of(m) == ENPASSANT)
+       if (type_of(m) == EN_PASSANT)
            board[capsq] = NO_PIECE;
  
        // Update material hash key and prefetch access to materialTable
    // If the moving piece is a pawn do some special extra work
    if (type_of(pc) == PAWN)
    {
-       // Set en-passant square if the moved pawn can be captured
+       // Set en passant square if the moved pawn can be captured
        if (   (int(to) ^ int(from)) == 16
            && (pawn_attacks_bb(us, to - pawn_push(us)) & pieces(them, PAWN)))
        {
@@@ -936,7 -934,7 +932,7 @@@ void Position::undo_move(Move m) 
        {
            Square capsq = to;
  
-           if (type_of(m) == ENPASSANT)
+           if (type_of(m) == EN_PASSANT)
            {
                capsq -= pawn_push(us);
  
@@@ -1015,7 -1013,7 +1011,7 @@@ void Position::do_null_move(StateInfo& 
    }
  
    st->key ^= Zobrist::side;
-   prefetch(TT.first_entry(st->key));
+   prefetch(TT.first_entry(key()));
  
    ++st->rule50;
    st->pliesFromNull = 0;
@@@ -1040,7 -1038,7 +1036,7 @@@ void Position::undo_null_move() 
  
  /// Position::key_after() computes the new hash key after the given move. Needed
  /// for speculative prefetch. It doesn't recognize special moves like castling,
- /// en-passant and promotions.
+ /// en passant and promotions.
  
  Key Position::key_after(Move m) const {
  
@@@ -1065,7 -1063,7 +1061,7 @@@ bool Position::see_ge(Move m, Value thr
  
    assert(is_ok(m));
  
-   // Only deal with normal moves, assume others pass a simple see
+   // Only deal with normal moves, assume others pass a simple SEE
    if (type_of(m) != NORMAL)
        return VALUE_ZERO >= threshold;
  
@@@ -1342,3 -1340,5 +1338,5 @@@ bool Position::pos_is_ok() const 
  
    return true;
  }
+ } // namespace Stockfish
diff --combined src/syzygy/tbprobe.cpp
index 28b70a4a10714e825be6c23f893dc570daf508da,5cfd38e5577feb8087eb37eff4d3458b92dee1de..c0cd04c1d4d8bd8f30607b616ae02363cb84da69
@@@ -1,6 -1,6 +1,6 @@@
  /*
    Stockfish, a UCI chess playing engine derived from Glaurung 2.1
-   Copyright (C) 2004-2020 The Stockfish developers (see AUTHORS file)
+   Copyright (C) 2004-2021 The Stockfish developers (see AUTHORS file)
  
    Stockfish is free software: you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
  #include <windows.h>
  #endif
  
- using namespace Tablebases;
+ using namespace Stockfish::Tablebases;
  
- int Tablebases::MaxCardinality;
+ int Stockfish::Tablebases::MaxCardinality;
+ namespace Stockfish {
  
  namespace {
  
@@@ -74,7 -76,7 +76,7 @@@ int MapB1H1H7[SQUARE_NB]
  int MapA1D1D4[SQUARE_NB];
  int MapKK[10][SQUARE_NB]; // [MapA1D1D4][SQUARE_NB]
  
 -int Binomial[6][SQUARE_NB];    // [k][n] k elements from a set of n elements
 +int Binomial[7][SQUARE_NB];    // [k][n] k elements from a set of n elements
  int LeadPawnIdx[6][SQUARE_NB]; // [leadPawnsCnt][SQUARE_NB]
  int LeadPawnsSize[6][4];       // [leadPawnsCnt][FILE_A..FILE_D]
  
@@@ -1000,7 -1002,7 +1002,7 @@@ uint8_t* set_sizes(PairsData* d, uint8_
      // so that d->lowestSym[i] >= d->lowestSym[i+1] (when read as LittleEndian).
      // Starting from this we compute a base64[] table indexed by symbol length
      // and containing 64 bit values so that d->base64[i] >= d->base64[i+1].
-     // See http://www.eecs.harvard.edu/~michaelm/E210/huffman.pdf
+     // See https://en.wikipedia.org/wiki/Huffman_coding
      for (int i = d->base64.size() - 2; i >= 0; --i) {
          d->base64[i] = (d->base64[i + 1] + number<Sym, LittleEndian>(&d->lowestSym[i])
                                           - number<Sym, LittleEndian>(&d->lowestSym[i + 1])) / 2;
@@@ -1141,7 -1143,7 +1143,7 @@@ void* mapped(TBTable<Type>& e, const Po
      if (e.ready.load(std::memory_order_acquire))
          return e.baseAddress; // Could be nullptr if file does not exist
  
-     std::unique_lock<std::mutex> lk(mutex);
+     std::scoped_lock<std::mutex> lk(mutex);
  
      if (e.ready.load(std::memory_order_relaxed)) // Recheck under lock
          return e.baseAddress;
@@@ -1321,7 -1323,7 +1323,7 @@@ void Tablebases::init(const std::string
      Binomial[0][0] = 1;
  
      for (int n = 1; n < 64; n++) // Squares
 -        for (int k = 0; k < 6 && k <= n; ++k) // Pieces
 +        for (int k = 0; k < 7 && k <= n; ++k) // Pieces
              Binomial[k][n] =  (k > 0 ? Binomial[k - 1][n - 1] : 0)
                              + (k < n ? Binomial[k    ][n - 1] : 0);
  
@@@ -1440,7 -1442,7 +1442,7 @@@ WDLScore Tablebases::probe_wdl(Position
  // If n = 100 immediately after a capture or pawn move, then the position
  // is also certainly a win, and during the whole phase until the next
  // capture or pawn move, the inequality to be preserved is
- // dtz + 50-movecounter <= 100.
+ // dtz + 50-move-counter <= 100.
  //
  // In short, if a move is available resulting in dtz + 50-move-counter <= 99,
  // then do not accept moves leading to dtz + 50-move-counter == 100.
@@@ -1610,3 -1612,5 +1612,5 @@@ bool Tablebases::root_probe_wdl(Positio
  
      return true;
  }
+ } // namespace Stockfish
diff --combined src/ucioption.cpp
index 7123915933b90f59d48627cd95d42dc809b07a38,d59c010017eee517dfa5f56dac1c2dbcebdcb423..5889ef6bd80e7d5ab8e69cddb1c6df0678404f8a
@@@ -1,6 -1,6 +1,6 @@@
  /*
    Stockfish, a UCI chess playing engine derived from Glaurung 2.1
-   Copyright (C) 2004-2020 The Stockfish developers (see AUTHORS file)
+   Copyright (C) 2004-2021 The Stockfish developers (see AUTHORS file)
  
    Stockfish is free software: you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
  #include "thread.h"
  #include "tt.h"
  #include "uci.h"
 +#include "hashprobe.h"
  #include "syzygy/tbprobe.h"
  
  using std::string;
  
+ namespace Stockfish {
  UCI::OptionsMap Options; // Global object
 +std::unique_ptr<HashProbeThread> hash_probe_thread;
  
  namespace UCI {
  
@@@ -45,13 -45,6 +47,13 @@@ void on_threads(const Option& o) { Thre
  void on_tb_path(const Option& o) { Tablebases::init(o); }
  void on_use_NNUE(const Option& ) { Eval::NNUE::init(); }
  void on_eval_file(const Option& ) { Eval::NNUE::init(); }
 +void on_rpc_server_address(const Option& o) {
 +      if (hash_probe_thread) {
 +              hash_probe_thread->Shutdown();
 +      }
 +      std::string addr = o;
 +      hash_probe_thread.reset(new HashProbeThread(addr));
 +}
  
  /// Our case insensitive less() function as required by UCI protocol
  bool CaseInsensitiveLess::operator() (const string& s1, const string& s2) const {
@@@ -90,7 -83,6 +92,7 @@@ void init(OptionsMap& o) 
    o["SyzygyProbeLimit"]      << Option(7, 0, 7);
    o["Use NNUE"]              << Option(true, on_use_NNUE);
    o["EvalFile"]              << Option(EvalFileDefaultName, on_eval_file);
 +  o["RPCServerAddress"]      << Option("<empty>", on_rpc_server_address);
  }
  
  
@@@ -200,3 -192,5 +202,5 @@@ Option& Option::operator=(const string
  }
  
  } // namespace UCI
+ } // namespace Stockfish