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

[Steve Borho]

On 01/02, 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))
> +        {

our minimum CTU size is 16x16, this seems an unnecessary check

> +            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

-- 
Steve Borho