> > In the Patch I see the line > CPU_CFLAGS=" -march=athlon64 -funroll-loops -minline-all-stringops > -ffast-math -mmmx -msse -msse2 > - -msse3 -m3dnow -DHAVE_MMX -DUSE_MMX -DHAVE_3dnow $CPU_CFLAGS" > > This is a lot of redundant and superfluos information. As far as I am > informed, > the only thing you *really* should switch on for AMD64 is the switch -msse3
Note that not every amd64 support sse3, mine for example doesn't. Note also that x86_64 *DOES NOT* mean amd64, that can also mean core 2 duo, which afaik doesnt support 3dnow. Also, are you sure that -ffast-math is safe here ? I dont like autodecting cpu at configure time because that, most of the time, leads to headaches. Why not honouring CFLAGS variable and that's all ? After all, that's the person who compiles that might know best what his system is. -DHAVE_MMX -DUSE_MMX -DHAVE_3dnow are some defines used in some source files to enable or not asm optimizations. I had tried that once on my amd64 but those were either useless (didnt enable anything at all, that was probably some old switchs that are unused now) or didn't compile (probably because it's nasm asm that is for x86_32). You can grep for them to see where they are used and what for ;) Regards, Alexis.
pgpUeOVpKB1Um.pgp
Description: PGP signature
