[x265] [PATCH] SAO_EO_2: sign asm code integration

[chen]

Oh, don't queue it, he forgot check bottom edge.


At 2015-01-03 14:12:54,"Steve Borho" <steve at borho.org> wrote:
>On 01/02, chen wrote:
>> ctuHeight unalign path
>
>I see, this code handles partial CTUs at the right edge, and thus it can
>be smaller.  I've queued the series.
>
>> At 2015-01-02 20:06:12,praveen at multicorewareinc.com wrote:
>> ># HG changeset patch
>> ># User Praveen Tiwari
>> ># Date 1420200336 -19800
>> ># Node ID 3de607b1444b0accf78a554ef2c1226b7cb13e5b
>> ># Parent  1629b8b3c5961dd50f458d44a30ae7c90fd15898
>> >SAO_EO_2: sign asm code integration
>> >
>> >diff -r 1629b8b3c596 -r 3de607b1444b source/encoder/sao.cpp
>> >--- a/source/encoder/sao.cpp	Wed Dec 24 18:27:46 2014 +0530
>> >+++ b/source/encoder/sao.cpp	Fri Jan 02 17:35:36 2015 +0530
>> >@@ -358,8 +358,29 @@
>> >         if (!tpely)
>> >             rec += stride;
>> > 
>> >-        for (x = startX; x < endX; x++)
>> >-            upBuff1[x] = signOf(rec[x] - tmpU[x - 1]);
>> >+        if (!(ctuWidth & 15))
>> >+        {
>> >+            pixel firstPxl, lastPxl;
>> >+
>> >+            if (!lpelx)
>> >+                firstPxl = upBuff1[0];
>> >+
>> >+            if (rpelx == picWidth)
>> >+                lastPxl = upBuff1[ctuWidth - 1];
>> >+
>> >+            primitives.sign(upBuff1, rec, &tmpU[- 1], ctuWidth);
>> >+
>> >+            if (!lpelx)
>> >+                upBuff1[0] = firstPxl;
>> >+
>> >+            if (rpelx == picWidth)
>> >+                upBuff1[ctuWidth - 1] = lastPxl;
>> >+        }
>> >+        else
>> >+        {
>> >+            for (x = startX; x < endX; x++)
>> >+                upBuff1[x] = signOf(rec[x] - tmpU[x - 1]);
>> >+        }
>> > 
>> >         for (y = startY; y < endY; y++)
>> >         {
>> >_______________________________________________
>> >x265-devel mailing list
>> >x265-devel at videolan.org
>> >https://mailman.videolan.org/listinfo/x265-devel
>
>> _______________________________________________
>> x265-devel mailing list
>> x265-devel at videolan.org
>> https://mailman.videolan.org/listinfo/x265-devel
>
>
>-- 
>Steve Borho
>_______________________________________________
>x265-devel mailing list
>x265-devel at videolan.org
>https://mailman.videolan.org/listinfo/x265-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20150104/f8ec43b3/attachment.html>