Dear Kurt,
I already saw, that stockfish was not building on alpha. Now I see, that it is
not building on all of the non i386 archs.
I am building stockfish on i386 and amd64 wthout problems. Exactly for building
on all arcitectures there is my patch: remove_msse_compile_opt.patch
But to my surprise it is NOT used on all archs.
I am new to the project and do (by now) not know how to connect to buildd
machines with other architectures. I will have to spend some time on this
problem now....
Regards,
Oliver
Am Samstag 20 Februar 2010 16:16:22 schrieb Kurt Roeckx:
[...]
> > /usr/bin/fakeroot debian/rules clean
> > dh clean --sourcedirectory=src --with quilt
> > dh_testdir -O--sourcedirectory=src
> > debian/rules override_dh_auto_clean
On exactly this stage quilt should be pacthing the Makefile and tt.cpp. This is
not done on other architectures than i386 and amd64. This is strange.
> > make[1]: Entering directory
> > `/build/buildd-stockfish_1.6.3JA-1-alpha-BXDLw6/stockfish-1.6.3JA' cd src
> > && make clean
> > make[2]: Entering directory
> > `/build/buildd-stockfish_1.6.3JA-1-alpha-BXDLw6/stockfish-1.6.3JA/src'
> > Makefile:314: .depend: No such file or directory
> > g++ -msse -MM application.cpp bitboard.cpp pawns.cpp material.cpp
> > endgame.cpp evaluate.cpp main.cpp misc.cpp move.cpp movegen.cpp
> > history.cpp movepick.cpp search.cpp piece.cpp position.cpp direction.cpp
> > tt.cpp value.cpp uci.cpp ucioption.cpp mersenne.cpp book.cpp bitbase.cpp
> > san.cpp benchmark.cpp > .depend cc1plus: error: unrecognized command line
> > option "-msse"
> > cc1plus: error: unrecognized command line option "-msse"
> > cc1plus: error: unrecognized command line option "-msse"
>
> [...]
Author: Oliver Korff <o...@xynyx.de>
Description: Thanks to Joona Kiiski I could remove the -msse option from the
gcc flags and stockfish might now build on other architectures.
Problem was the include of xmmintrin.h, that holds the SSE extensions.
Marc Costelba: "in any case is only a speed optimization, not a
functional change." This was a FTBS Bug for some architectures: (
Closes: 559724) Upstream is not intersted to keep portability on
every price.
Index: stockfish-1.6.2JA/src/Makefile
===================================================================
--- stockfish-1.6.2JA.orig/src/Makefile 2010-01-06 12:56:33.000000000 +0100
+++ stockfish-1.6.2JA/src/Makefile 2010-01-06 12:57:24.000000000 +0100
@@ -26,7 +26,7 @@
### Compiler speed switches for both GCC and ICC. These settings are generally
### fast on a broad range of systems, but may be changed experimentally
### ==========================================================================
-GCCFLAGS = -O3 -msse
+GCCFLAGS = -O3
ICCFLAGS = -fast -msse
ICCFLAGS-OSX = -fast -mdynamic-no-pic
@@ -301,6 +301,6 @@
### Dependencies. Do not change
.depend:
- $(CXX) -msse -MM $(OBJS:.o=.cpp) > $@
+ $(CXX) -MM $(OBJS:.o=.cpp) > $@
include .depend
Index: stockfish-1.6.2JA/src/tt.cpp
===================================================================
--- stockfish-1.6.2JA.orig/src/tt.cpp 2010-01-06 12:53:53.000000000 +0100
+++ stockfish-1.6.2JA/src/tt.cpp 2010-01-06 12:56:19.000000000 +0100
@@ -25,7 +25,7 @@
#include <cassert>
#include <cmath>
#include <cstring>
-#include <xmmintrin.h>
+// #include <xmmintrin.h>
#include "movegen.h"
#include "tt.h"
@@ -178,9 +178,9 @@
__asm__ ("");
#endif
- char const* addr = (char*)first_entry(posKey);
- _mm_prefetch(addr, _MM_HINT_T2);
- _mm_prefetch(addr+64, _MM_HINT_T2); // 64 bytes ahead
+// char const* addr = (char*)first_entry(posKey);
+// _mm_prefetch(addr, _MM_HINT_T2);
+// _mm_prefetch(addr+64, _MM_HINT_T2); // 64 bytes ahead
}
