Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SF failing to compile with MSVC2013 32-bit #178

Closed
zamar opened this issue Dec 30, 2014 · 3 comments
Closed

SF failing to compile with MSVC2013 32-bit #178

zamar opened this issue Dec 30, 2014 · 3 comments

Comments

@zamar
Copy link

zamar commented Dec 30, 2014

Syed Fahad reported in Talkchess that SF does not compile on MSVC 2013 32-bit, because _BitScanForward64 etc. are missing:
http://talkchess.com/forum/viewtopic.php?t=54798

Checking from the MSVC documentation, this intrinsic is not supported in 32-bit mode:
http://msdn.microsoft.com/en-us/library/wfd9z0bb.aspx

@mcostalba
Copy link

The problem is that -DUSE_BSFQ is used even in 32 bit case, while it should be used only for 64 bit as written in the Makefile:

bsfq = yes/no --- -DUSE_BSFQ --- Use bsfq x86_64 asm-instruction (only with GCC and ICC 64-bit)

If standard make command is used (withouth additional flags) this should not happen because bsfq flag in Makefile is enabled only for 64 bit architectures.

@mcostalba
Copy link

Syed confirmed on the same forum thread that SF compiles just fine and this is a feature request and not a bug report. He was not clear in his first post.

The supposed feature is to introduce platform specific code to support hardware bitscan on 32 bit under MSVC. This is not a feature because sofwtare bitscan is very fast anyhow and we should avoid to introduce unnecessary platform specific code.

So I suggest to close this as non-issue.

@zamar
Copy link
Author

zamar commented Dec 31, 2014

OK. Thanks for the investigation Marco!

This is a non-issue, as we have no plans to support BSFQ on Win32.

closing the issue

@zamar zamar closed this as completed Dec 31, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants