2012/12/7 Justin Ruggles <[email protected]>:
>> +cglobal sbr_qmf_deint_neg, 2,3,4,v,src,vrev
>
> needs INIT_XMM sse

Is one needed for each cglobal? Because there's already one prior to
that function, and no other one.

> I don't quite get the use of m2 here. It looks unnecessary.

Indeed, but this patch is kind of "sent without proper checking",
because I couldn't find a sample for it, even in the fate suite. Sorry
about the ping, I could have just dropped it.

>> +    jl      .loop
>
> Where is the return?

And indeed it is missing, because this code is never run.

Unless someone can point a sample, it is probably not worth working on
that patch.

-- 
Christophe
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to