[FFmpeg-cvslog] r21277 - trunk/configure

[Måns Rullgård]

Dominik 'Rathann' Mierzejewski <dominik at rangers.eu.org> writes:

> On Monday, 18 January 2010 at 02:25, mru wrote:
>> Author: mru
>> Date: Mon Jan 18 02:25:57 2010
>> New Revision: 21277
>> 
>> Log:
>> Only set HAVE_MMX when a supported CPU is specified
>> 
>> Modified:
>>    trunk/configure
>> 
>> Modified: trunk/configure
>> ==============================================================================
>> --- trunk/configure	Mon Jan 18 01:34:28 2010	(r21276)
>> +++ trunk/configure	Mon Jan 18 02:25:57 2010	(r21277)
>> @@ -1789,8 +1789,12 @@ case $cpu in
>>          cpuflags="-mcpu=cell"
>>          enable ldbrx
>>      ;;
>> +    i[345]86|pentium)
>> +        cpuflags="-march=$cpu"
>> +        disable mmx
>> +    ;;
>>      # targets that do NOT support conditional mov (cmov)
>> -    i[345]86|pentium|pentium-mmx|k6|k6-[23]|winchip-c6|winchip2|c3)
>> +    pentium-mmx|k6|k6-[23]|winchip-c6|winchip2|c3)
>>          cpuflags="-march=$cpu"
>>          disable cmov
>>      ;;
>
> Um... Am I seeing this wrong or did you just enable cmov in the non-mmx case?

No, cmov is off by default.  Those "disable cmov" are redundant.

-- 
M?ns Rullg?rd
mans at mansr.com