[Ffmpeg-devel] [PATCH] Avoid generating MMX2 code for fft_3dn2.c

[Guillaume POIRIER]

Hi,

Zuxy Meng a ?crit :
> 2006/5/18, Reimar D?ffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de>:
> 
>> Hi,
>> On Thu, May 18, 2006 at 08:38:55PM +0800, Zuxy Meng wrote:
>> > --- ffmpeg/libavcodec/Makefile        2006-05-18 01:39:28.000000000
>> +0800
>> > +++ ffmpeg.new/libavcodec/Makefile    2006-05-18 20:09:10.000000000
>> +0800
>> > @@ -351,7 +351,7 @@
>> >  ifdef TARGET_BUILTIN_3DNOW
>> >  i386/fft_3dn.o: CFLAGS+= -m3dnow
>> >  ifeq ($(TARGET_ARCH_X86),yes)
>> > -i386/fft_3dn2.o: CFLAGS+= -march=athlon
>> > +i386/fft_3dn2.o: CFLAGS+= -march=athlon -mno-sse
>>
>> Huh? I really think the original athlons did not support sse, no
>> -march=athlon can't include -msse, can it??
>>
> 
> -mno-sse to exclude mmx2 code, like prefetcht1:-)


Though this is ugly, I don't see how it can be better done (there's no
-march=k6-2+, etc...).

Reimar did react to this patch, but as your patch seem to be correct,
I'll apply it unless Reimar shouts out.

Guillaume