Have you verified is a non functional change (it is not written in the
commit logs)

Have you done some speed tests?

On Fri, Jan 8, 2016 at 12:56 AM, VoyagerOne notifications@github.com
wrote:

For enpassants the piece_on(to_sq) will be NONE.
So it will always skip the early return and run the very expensive see()
call.
This tweak will make sure enpassants return early, which should prove to

be a small speed up.

You can view, comment on, or merge this pull request online at:

#563
Commit Summary

• SEE

File Changes

• M src/position.cpp
  https://github.com/official-stockfish/Stockfish/pull/563/files#diff-0
  (2)

Patch Links:

• https://github.com/official-stockfish/Stockfish/pull/563.patch
• https://github.com/official-stockfish/Stockfish/pull/563.diff

—
Reply to this email directly or view it on GitHub
#563.

The bench of VoyagerOne's patch is 8603285 It's the same as master so there's most likely no functional change

@VoyagerOne:

But on the other hand, en passant capture are very rare.

Personally I wouldn't expect any speed gains on this change. Please provide some benchmarks to support your patch.

@zamar
I am not sure the best way to test for speed.
So you can just close this request if you like.

@VoyagerOne
https://github.com/zardav/FishBench

@VoyagerOne:

It's up to you.

• If you want to provide evidence for your patch, please do so.
• If you don't feel like it, then just close the pull request...

@zamar
Ok I will close it.

BTW, if you are around...I have a patch that needs approval.