[FFmpeg-cvslog] r21736 - trunk/libavcodec/x86/dsputil_mmx.c

David Conrad lessen42
Wed Feb 10 23:08:07 CET 2010


On Feb 10, 2010, at 4:16 PM, Alex Converse wrote:

> On Tue, Feb 9, 2010 at 9:02 PM, conrad <subversion at mplayerhq.hu> wrote:
>> Author: conrad
>> Date: Wed Feb 10 03:02:06 2010
>> New Revision: 21736
>> 
>> Log:
>> Enable SSE2 (put|avg)_pixels_16_sse2
>> 
>> SVQ1 chroma has been special-cased aligned to 16-bytes since at least r15466
>> Other architectures also assume 16-byte alignment here too but set STRIDE_ALIGN
>> to 16.
>> 
>> Modified:
>>   trunk/libavcodec/x86/dsputil_mmx.c
>> 
> 
> This broke vp6f:
> http://fate.multimedia.cx/index.php?build_record=178551
> http://fate.multimedia.cx/index.php?build_record=178488

Looks like vp5 and vp6 use width 16 pixel functions on chroma as well. I'd really prefer if SSE used 16-byte aligment like other architectures (first patch). Otherwise VP5/6 need to be special-cased as well (second patch), and potentially other codecs that FATE doesn't test, which gets extremely ugly fast.

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: ffmpeg-sse-align16.txt
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20100210/60332e5f/attachment.txt>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: ffmpeg-vp56-specialcase.txt
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20100210/60332e5f/attachment-0001.txt>



More information about the ffmpeg-cvslog mailing list