[FFmpeg-devel] [PATCH] Make mmx2 dependent on mmx/ssse3 dependent on mmx2

[Panagiotis Issaris]

M?ns Rullg?rd schreef:
> Panagiotis Issaris <takis.issaris at uhasselt.be> writes:
> 
>> Hi,
>>
>> First of all, I'm not sure about this patch... The thing is, I had
>> expected the configure script to disable mmx2 and ssse3 too when
>> disabling mmx, which it currently doesn't do. But, that's just because
>> I'm not aware of any machines implementing mmx2 but not mmx. Anyway,
>> attached patch makes mmx2 dependent on mmx, likewise for ssse3 which
>> becomes dependent on mmx2. So, with the attached patch --disable-mmx
>> disables them all.
>>
>> Index: configure
>> ===================================================================
>> --- configure	(revision 12527)
>> +++ configure	(working copy)
>> @@ -782,8 +784,8 @@
>>  iwmmxt_deps="armv4l"
>>  mmi_deps="mips"
>>  mmx_deps="x86"
>> -mmx2_deps="x86"
>> -ssse3_deps="x86"
>> +mmx2_deps="x86 mmx"
>> +ssse3_deps="x86 mmx2"
>>  vis_deps="sparc"
>>
>>  # decoders / encoders
> 
> I can imagine someone wishing to disable only mmx on an sse-capable
> machine to save some space.  If you want an option to disable all
> assembler optimisations, it should be called something else.

Yes, you're both right. How about something like the attached patch?



With friendly regards,
Takis
--
vCard: http://issaris.org/pi.vcf
PGP key: http://issaris.org/pi.key
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: pi-20080321T121400-ffmpeg-disable_simd.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080321/09c821de/attachment.txt>