From: Joost VandeVondele <Joost.VandeVondele@gmail.com>
Date: Sat, 29 Apr 2017 03:27:39 +0000 (-0700)
Subject: Copy killers in the movepicker
X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=b1b19343cd1f5ec65084dc11a0a0b4d5ece2a24b;hp=b1b19343cd1f5ec65084dc11a0a0b4d5ece2a24b

Copy killers in the movepicker

ss->killers can change while the movepicker is active.
The reason ss->killers changes is related to the singular
extension search in the moves loop that calls search<>
recursively with ss instead of ss+1,
effectively using the same stack entry for caller and callee.
By making a copy of the killers,
the movepicker does the right thing nevertheless.

Tested as a bug fix

STC:
http://tests.stockfishchess.org/tests/view/58ff130f0ebc59035df33f37
LLR: 2.95 (-2.94,2.94) [-3.00,1.00]
Total: 70845 W: 12752 L: 12716 D: 45377

LTC:
http://tests.stockfishchess.org/tests/view/58ff48000ebc59035df33f3d
LLR: 2.96 (-2.94,2.94) [-3.00,1.00]
Total: 28368 W: 3730 L: 3619 D: 21019

Bench: 6465887

Closes #1085
---