[FFmpeg-devel] [PATCH] move vp3dsp to yasm

Ronald S. Bultje rsbultje
Sun Aug 29 01:10:49 CEST 2010


Hi,

On Fri, Aug 27, 2010 at 10:32 AM, Ronald S. Bultje <rsbultje at gmail.com> wrote:
> 2010/8/27 M?ns Rullg?rd <mans at mansr.com>:
>> "Ronald S. Bultje" <rsbultje at gmail.com> writes:
>>> On Thu, Aug 26, 2010 at 5:46 PM, Ronald S. Bultje <rsbultje at gmail.com> wrote:
>>>> On Thu, Aug 26, 2010 at 5:20 PM, Ronald S. Bultje <rsbultje at gmail.com> wrote:
>>>>> $subj. Fixes part of the VP3/5/6-fate-Win64 issues. The last part
>>>>> (h264_chroma_mc8) is yet to be done. Passes make fate-vp3 with both
>>>>> sse2 disabled as well as enabled.
>>>>
>>>> This one has the obvious "??" replaced with correct code on x86-32
>>>> also (thanks David!).
>>>
>>> Actually tested on x86-32 also now, and passes make fate-vp3. Had
>>> already tested and confirmed that on x86-64 for mmx and sse2 code.
>>>
>>> - ? ? ? ? ? ? ? ?if(0){//mm_flags & FF_MM_SSE2){
>>> + ? ? ? ? ? ? ? ?if(mm_flags & FF_MM_SSE2){
>>
>> Curious...
>
> Hm, quilt messed up, it thought it was if(0) before my patch series,
> fixed locally...

Now with another fix that makes it work on Win64 also. FATE passes on
OSX x86-32/64 and FATE vp3 and vp6 tests pass on Win64. This should be
safe to apply, so I'll apply in 3 days unless people object.

Ronald
-------------- next part --------------
A non-text attachment was scrubbed...
Name: vp3dsp-yasm.patch
Type: application/octet-stream
Size: 55812 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100828/b66a8a52/attachment.obj>



More information about the ffmpeg-devel mailing list