Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Reduce cache impact of between_bb #37

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Contributor

glinscott commented Dec 3, 2012

This is a small speed win on my system (Linux, i7). Idea from Gerd Isenberg here: http://www.talkchess.com/forum/viewtopic.php?t=46240.

Best of 10 bench runs after:
Total time (ms) : 2942
Nodes searched : 5695710
Nodes/second : 1935999

Best of 10 bench runs before:
Total time (ms) : 2968
Nodes searched : 5695710
Nodes/second : 1919039

Owner

mcostalba commented Dec 4, 2012

Thanks Gary!

I'd like to look at this patch deeper (probably this week-end) before to merge.

Contributor

glinscott commented Dec 4, 2012

I'm not 100% sure it will be a speed win on all systems, since it's so small on my system. I tried a few other variations (using the 240 size table, trying rotate left instead of the swap), and this ended up the fastest.

@mcostalba mcostalba closed this Aug 31, 2015

alwey pushed a commit to alwey/Stockfish that referenced this pull request Nov 23, 2016

Merge pull request #37 from ianfab/master
Fix Antichess en passant move generation bug
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment