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

Jason Garrett-Glaser darkshikari
Tue Aug 24 21:44:24 CEST 2010


On Tue, Aug 24, 2010 at 3:39 PM, Ivan Kalvachev <ikalvachev at gmail.com> wrote:
> 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.

This sounds like a perfectly good time to dump support for pre-MMX
CPUs when --disable-mmx isn't used.

Dark Shikari



More information about the ffmpeg-devel mailing list