On Sat, 2007-02-24 at 14:21 +0100, Ichthyostega wrote: > -----BEGIN PGP SIGNED MESSAGE----- > Hash: SHA1 > > Johannes Sixt schrieb: > > - Adding all those optimization options, -funroll-loops > > -minline-all-stringops > > etc. to the default compile line is a big NO-NO. It may be convenient for > > you, but it is not good practice in general. Is there any harm done if I > > remove them? > > Typically and in most cases, it is just sufficient to use -O3 > The compiler has sensible defaults for most cases. > > 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 > This includes all previous SSE levels and superseedes MMX and 3DNow!
well probably true, but as far as i remember i did some tests and found out that -f options also did some good at least for pentimm, though i don't know now if they are automatically enabled by -o3 ... btw: gcc discourage use of -O3 since it is not tested well enough to be sure it does not have any bugs. > So, at least the line should be for X86_64: > CPU_CFLAGS=" -march=athlon64 -msse3 .......... > > What are the "-DHAVE_MMX -DUSE_MMX -DHAVE_3dnow" good for? > Does anyone know? they are for ifdefs in the code, to make it easy to check wheather the functionality mentioned is supported and should be compiled. general rule: if you don't know exactly what it does, leave it alone bye andraz _______________________________________________ Cinelerra mailing list [email protected] https://init.linpro.no/mailman/skolelinux.no/listinfo/cinelerra
