Hi, On Sat, Dec 27, 2014 at 12:55 PM, Clément Bœsch <u...@pkh.me> wrote:
> On Sat, Dec 27, 2014 at 11:02:42AM -0500, Ronald S. Bultje wrote: > > The value is not used outside the branch. > > --- > > libavcodec/x86/vp9lpf.asm | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/libavcodec/x86/vp9lpf.asm b/libavcodec/x86/vp9lpf.asm > > index e117798..15d4159 100644 > > --- a/libavcodec/x86/vp9lpf.asm > > +++ b/libavcodec/x86/vp9lpf.asm > > @@ -469,8 +469,8 @@ cglobal vp9_loop_filter_%1_%2_16, 2, 6, 16, %3 + %4, > dst, stride, mstride, dst2, > > > > ; (m3: fm, m8..15: p3 p2 p1 p0 q0 q1 q2 q3) > > ; calc flat8in (if not 44_16) and hev masks > > - mova m6, [pb_81] ; [1 1 1 1 ...] > ^ 0x80 > > %if %2 != 44 > > + mova m6, [pb_81] ; [1 1 1 1 ...] > ^ 0x80 > > ABSSUB_GT m2, m8, m11, m6, m5 ; abs(p3 - p0) > <= 1 > > mova m8, [pb_80] > > ABSSUB_GT m1, m9, m11, m6, m5, m8 ; abs(p2 - p0) > <= 1 > > Should this be merged or moved just after patch 4? Well, it's related to patch 4 and 5, but somewhat orthogonal. I'll keep it separate and put it in that sequence (6 is indeed completely different so it should go before that one). Re-arranged patch queue on github. Ronald _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
