Qsearch simplification. (#1828)
authorJörg Oster <osterj165@googlemail.com>
Sun, 25 Nov 2018 10:27:40 +0000 (11:27 +0100)
committerMarco Costalba <mcostalba@users.noreply.github.com>
Sun, 25 Nov 2018 10:27:40 +0000 (11:27 +0100)
commit6ab92d2e1c417fa7a9fd87eee0add2bf02112296
tree05a4758c1cee39218af7300edbf80550cfd6ba2b
parentbb58bc215c93a17cd6ee672eac5495b48e9f3d22
Qsearch simplification. (#1828)

Don't do an extra TT update in case of a fail-high,
but simply break off the moves loop and let the TT update
at the end of qsearch do this job.
Same workflow/logic as in our main search function now.

Tested for no regression to be on the safe side.
STC
LLR: 2.96 (-2.94,2.94) [-3.00,1.00]
Total: 30237 W: 6665 L: 6560 D: 17012
http://tests.stockfishchess.org/tests/view/5bf928e80ebc5902bced3f3a

LTC
LLR: 2.95 (-2.94,2.94) [-3.00,1.00]
Total: 51067 W: 8625 L: 8553 D: 33889
http://tests.stockfishchess.org/tests/view/5bf937180ebc5902bced3fdc

No functional change.
src/search.cpp