Crafty v25.3 compile

[jhaglund2]

I'm back again trying to compile Crafty v25.3, with Microsoft VS 2017.

Warnings:
thread.c(455): warning C4028: formal parameter 1 different from declaration

Code: Select all

int ThreadSplit(TREE * tree, int ply, int depth, int alpha, int o_alpha, int done)

thread.c(888): warning C4028: formal parameter 1 different from declaration

Code: Select all

TREE *GetBlock(TREE * RESTRICT parent, int tid) {

book.c, line 1438 seems incorrect.

book.c(1438): error C2440: 'function': cannot convert from 'int (__fastcall *)(const void *,const void *)' to '_CoreCrtNonSecureSearchSortCompareFunction'

book.c(1438): warning C4024: 'qsort': different types for formal and actual parameter 4

Code: Select all

qsort((char *) buffer, number, sizeof(BB_POSITION), BookupCompare);

change: utility.c, line 2558

utility.c(2558): warning C4477: 'printf' : format string '%llu' requires an argument of type 'unsigned __int64', but variadic argument 1 has type 'DWORD'

Code: Select all

 printf("Current ideal CPU is %llu\n", dwCPU);

to

Code: Select all

 printf("Current ideal CPU is %lu\n", dwCPU);

change: utility.c, line 2591

Code: Select all

 printf("Starting thread on node " PRId64 " CPU mask %I64d\n", ulNumaNode,

to

Code: Select all

printf("Starting thread on node " PRId64 " CPU mask %ld\n", ulNumaNode,

tbcore.c(11): fatal error C1083: Cannot open include file: 'unistd.h': No such file or directory
This needs a compatibility fix.

Code: Select all

#include <unistd.h>

option.c: lines 922, 964, 1001

option.c(922): error C2708: '__builtin_clzll': actual parameters length in bytes differs from previous call or reference

Code: Select all

hash_table_size = ((1ull) << MSB(new_hash_size)) / 16;

option.c(964): error C2708: '__builtin_clzll': actual parameters length in bytes differs from previous call or reference

Code: Select all

hash_path_size = ((1ull) << MSB(new_hash_size / sizeof(HPATH_ENTRY)));

option.c(1001): error C2708: '__builtin_clzll': actual parameters length in bytes differs from previous call or reference

Code: Select all

pawn_hash_table_size =   1ull << MSB(new_hash_size / sizeof(PAWN_HASH_ENTRY));

chess.h: lines 324,-326

Code: Select all

#define PopCnt(v) __builtin_popcountll(v)
#define LSB(v)    __builtin_ctzll(v)
#define MSB(v)    (63 - __builtin_clzll(v))

These are causing the compiler to error.

I think working on MacOs, Linux, GCC only, has lost compile compatibility with Windows.

Please address a fix.
Thanks!

[jhaglund2]

Also:

error LNK2019: unresolved external symbol _sleep referenced in function _Iterate

iterate: line 182

Code: Select all

if (elo_sleep) sleep (time_limit/167);

Windows:

Code: Select all

if (elo_sleep) Sleep (time_limit/167);

[Dann Corbit]

Also:

error LNK2019: unresolved external symbol _sleep referenced in function _Iterate

iterate: line 182

Code: Select all

if (elo_sleep) sleep (time_limit/167);

Windows:

Code: Select all

if (elo_sleep) Sleep (time_limit/167);

Careful, they are not the same time units.

http://man7.org/linux/man-pages/man3/sleep.3.html
https://msdn.microsoft.com/en-us/library/ms918428.aspx

[jhaglund2]

https://en.wikipedia.org/wiki/Sleep_(system_call)

Code: Select all

C examples
In Windows OS:
Sleep(2*1000);  // Sleep for 2 seconds

In Unix:
sleep(2);       // Sleep for 2 seconds

[jdart]

Re the errors about unistd.h (tbcore.c(11) for example) - that doesn't really make sense to me. In the source I have that is not included if _WIN32 is defined, and MSVC should define that by default.

--Jon

[Sven]

Re the errors about unistd.h (tbcore.c(11) for example) - that doesn't really make sense to me. In the source I have that is not included if _WIN32 is defined, and MSVC should define that by default.

--Jon

Typically Crafty includes unistd.h and other POSIX headers within #if defined(UNIX).

[jhaglund2]

After looking deeper into the problem, and many trials and changes, I managed to get Crafty v25.3 compiled, with Microsoft VS 2017.
However, I couldn't get /DSYZYGY to work, so it is without tablebase support for now.

crafty.obj : error LNK2019: unresolved external symbol _tb_init_impl referenced in function _Option
crafty.obj : error LNK2019: unresolved external symbol _tb_probe_wdl_impl referenced in function _Search
crafty.obj : error LNK2019: unresolved external symbol _tb_probe_root_impl referenced in function _RootMoveList
crafty.obj : error LNK2019: unresolved external symbol _TB_LARGEST referenced in function _Option

Also, I had to comment out:
book.c, line 1438

book.c(1438): error C2440: 'function': cannot convert from 'int (__fastcall *)(const void *,const void *)' to '_CoreCrtNonSecureSearchSortCompareFunction'
book.c(1438): warning C4024: 'qsort': different types for formal and actual parameter 4

Code: Select all

//qsort((char *) buffer, number, sizeof(BB_POSITION), BookupCompare);

Compiled for up to 64 CPUs. Give it a whirl.
https://www.mediafire.com/file/7e27pikd ... y.zip/file

[jhaglund2]

Found the syntax error:

Code: Select all

qsort((char *) buffer, number, sizeof(BB_POSITION), BookupCompare);

to

Code: Select all

qsort((char *) buffer, number, sizeof(BB_POSITION), BookUpCompare);

Updated the file at the previous URL.

[jhaglund2]

By some miracle , I managed to (Microsoft VS 2017) compile with /DSYZYGY. It took a lot of changes. I'm not sure it even works as intended.

If someone could test it for me, that would be nice.

https://www.mediafire.com/file/p0vfoawy ... b.zip/file