On 09/11/14 12:06 AM, James Almer wrote:
> On 08/11/14 11:37 PM, Kieran Kunhya wrote:
>> From 196dd0d9eb40ca18e3bba993f5198bd1bde536ce Mon Sep 17 00:00:00 2001
>> From: Kieran Kunhya <kier...@ob-encoder.com>
>> Date: Sat, 8 Nov 2014 20:34:25 -0600
>> Subject: [PATCH] swscale: Enable yuv2planeX_8 assembly that had been disabled
>> for some reason
>>
>> ---
>> libswscale/x86/swscale.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/libswscale/x86/swscale.c b/libswscale/x86/swscale.c
>> index 8ce87b3..8a688b3 100644
>> --- a/libswscale/x86/swscale.c
>> +++ b/libswscale/x86/swscale.c
>> @@ -430,7 +430,7 @@ switch(c->dstBpc){ \
>> case 16: do_16_case;
>> break; \
>> case 10: if (!isBE(c->dstFormat)) vscalefn = ff_yuv2planeX_10_ ##
>> opt; break; \
>> case 9: if (!isBE(c->dstFormat)) vscalefn = ff_yuv2planeX_9_ ##
>> opt; break; \
>> - default: if (condition_8bit) /*vscalefn = ff_yuv2planeX_8_ ##
>> opt;*/ break; \
>> + default: if (condition_8bit) vscalefn = ff_yuv2planeX_8_ ##
>> opt; break; \
>> }
>> #define ASSIGN_VSCALE_FUNC(vscalefn, opt1, opt2, opt2chk) \
>> switch(c->dstBpc){ \
>> --
>> 1.7.9.5
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel@ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> This breaks ten filter-pixfmts FATE tests when using SSE2 or above (at least
> on
> x86_64), so i guess that's why it was disabled.
>
> Enabling the mmxext version until the rest are fixed is probably a better
> idea
> than disabling everything, though.
Nevermind, i thought it succeeded with the mmxext version but i didn't realize
that
it's only compiled on x86_32, so it of course ran with the pure c version
instead.
Tested now on x86_32 and the mmxext version is also broken.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
