[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Bug-gnubg] gcc 4.0 patch for 0.14.3
From: |
Russ Allbery |
Subject: |
Re: [Bug-gnubg] gcc 4.0 patch for 0.14.3 |
Date: |
Sat, 21 Jan 2006 16:40:56 -0800 |
User-agent: |
Gnus/5.110004 (No Gnus v0.4) XEmacs/21.4.17 (linux) |
Øystein Johansen <address@hidden> writes:
> At last I was able to look at this. No backgammon playing today. Just a
> lazy wednesday. Really good!
Sounds excellent. :) Sorry about the delay in responding; it's been a
busy week.
> This snip of code has never been compiled at my system since it's now
> inside a #if HAVE_MMAP && ! USE_SSE_VECTORIZE and of course I always use
> the vectorization code. The patch is commited.
Thanks!
>> --- gnubg-0.14.3.orig/rollout.c
>> +++ gnubg-0.14.3/rollout.c
>> @@ -219,7 +219,7 @@
>> nPermutationSeed = n;
>> }
>>
>> -static int nSkip;
>> +int nSkip;
> Why this? I don't get any errors or warnings with my gcc-4.0.2
In 0.14.3, rollout.h has "extern int nSkip" and rollout.c includes
rollout.h. A variable can't be declared static after being declared
extern, so this breaks compilation.
My guess is that this has already been fixed some other way in CVS.
> I really recommend trying a cvs build. Lots of things has been improved
> since 0.14.3. Among those things are the sse vectorization which
> increases the evaluation speed by about 50%.
Do you have an estimate for 0.15? While sometimes Debian does include CVS
snapshots of software, generally we prefer to use official upstream
releases for various reasons.
gnubg is a bit of a special case since it's generally beta software
anyway, but still, if what's in CVS is significantly better, I encourage
you to stick a version number on it! :)
--
Russ Allbery (address@hidden) <http://www.eyrie.org/~eagle/>