[FFmpeg-devel] [PATCH] Remove global mm_flags variable

Ramiro Polla ramiro.polla
Tue Aug 24 21:50:49 CEST 2010


2010/8/24 M?ns Rullg?rd <mans at mansr.com>:
> Ivan Kalvachev <ikalvachev at gmail.com> writes:
>> On 8/24/10, Mans Rullgard <mans at mansr.com> wrote:
>>> -
>>> -#define emms_c() \
>>> -{\
>>> - ? ?if (mm_flags & FF_MM_MMX)\
>>> - ? ? ? ?emms();\
>>> -}
>>> +#define emms_c() emms()
>>
>> Michael, are you really OK with that change?
>> The result would be illegal instruction on all pre-MMX CPUs.
>
> Yes, so? ?When was the last time you saw one of those. ?The one person
> still running such old hardware will have to use --disable-mmx now.

Then we better document it in the Changelog or on some other doc/ (or
on the main page, maybe with a funny note on how long it took us to do
it =)



More information about the ffmpeg-devel mailing list