Re: Some gnuchess fixes

[Lukas Geyer]

Hi,

new version up under

http://www.mathematik.uni-dortmund.de/lsix/geyer/gnuchess

This one should improve performance by really inlining the
performance-critical functions leadz() and nbits(). Furthermore, I have
protected other compilers than gcc from seeing __attribute__ at all, and
removed some useless inline definitions. (The linker can not inline
functions.) I suggest the following for inlining: Declare functions which
are only used in one source file as static inline, collect global inline
functions in src/inlines.h and protect the original definitions by some
#ifdef (currently #ifdef NO_INLINE, but this may change or be checked by
configure) for compilers which do not support inlining or people wishing
to compile without even minimal optimization.

I have not yet fixed the illegal moves in lost positions, but the code in
sort.c looks very suspicious, in particular the PhasePick() and
PhasePick1() functions which contain some switch/case blocks with
fallthrough semantics. I am going to have a closer look at them later.

Cheers,
Lukas