Imported to CVS, review requested: Glide3

[Hans de Goede]

Thanks for trying it out, x86_64 support is as said in the specfile 
untested.

The code in question is:
#elif defined(__GNUC__) && defined(__i386__)
/*
  * This is the __linux__ code.
  */
#define P6FENCE \
    asm("xchg %%eax, %0" : : "m" (_GlideRoot.p6Fencer) : "eax");
#else
#  error "P6 Fencing code needs to be added for this compiler"
#endif

Now the questions are:
-what symbol gets defined instead of __i386__ for x86_64
-is the asm code correct for x86_64

Regards,

Hans




Matthias Saou wrote:
> Hi,
> 
> I just tried to recompile it for FC Development x86_64 and got this error :
> 
> [...]
>  gcc -DX11 -fomit-frame-pointer -funroll-loops -fexpensive-optimizations -
> ffast-math -DBIG_OPT -Wall -DGLIDE_LIB -DGLIDE3 -DGLIDE3_ALPHA -DH3 -
> DGLIDE_PACKET3_TRI_SETUP=1 -DUSE_PACKET_FIFO=1 -DGLIDE_HW_TRI_SETUP=1 -
> DFX_GLIDE_NAPALM=1 -DFX_GLIDE_H5_CSIM=1 -DGLIDE_PLUG -DGLIDE_SPLASH -
> DFX_STATIC_BUILD -DGLIDE_INIT_HWC -DGLIDE_USE_C_TRISETUP -I../../../../h5/
> glide3/src -I../../../h5/incsrc -I../../../../h5/incsrc -I../../../../h5/
> minihwc -I. -I../../../../swlibs/fxmemmap -I../../../../swlibs/fxmisc -
> I../../../../swlibs/newpci/pcilib -I../../../../swlibs/texus2/lib -O2 -g -
> pipe -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -m64 -mtune=nocona -O6 -Wno-
> unknown-pragmas -Wno-unused-value -Wno-unused -Wno-missing-noreturn -Wp,-
> MD,.deps/gsplash.pp -c ../../../../h5/glide3/src/gsplash.c  -fPIC -DPIC -
> o .libs/gsplash.o In file included from ../../../../h5/glide3/src/
> gsplash.c:170: ../../../../h5/glide3/src/fxglide.h:2104:4: error: #error
> "P6 Fencing code needs to be added for this compiler" make[3]: ***
> [gsplash.lo] Error 1 make[3]: Leaving directory `/usr/src/rpm/BUILD/Glide3/
> build-5/h5/glide3/src'
> 
> Not sure what should be done...
> 
> Matthias
>