[x265] [PATCH] TComYuv.cpp, use new luma_copy_ps asm primitives where feasible

[Steve Borho]

On Tue, Nov 12, 2013 at 6:28 AM, <praveen at multicorewareinc.com> wrote:

> # HG changeset patch
> # User Praveen Tiwari
> # Date 1384259293 -19800
> # Node ID 9836ede305fad313f4282063887cc9d028c435a8
> # Parent  694c0d3db925d4d0f8311a1e5a0a83b53ab99af7
> TComYuv.cpp, use new luma_copy_ps asm primitives where feasible
>
> diff -r 694c0d3db925 -r 9836ede305fa source/Lib/TLibCommon/TComYuv.cpp
> --- a/source/Lib/TLibCommon/TComYuv.cpp Tue Nov 12 17:41:21 2013 +0530
> +++ b/source/Lib/TLibCommon/TComYuv.cpp Tue Nov 12 17:58:13 2013 +0530
> @@ -281,7 +281,8 @@
>      uint32_t  srcstride = getStride();
>      uint32_t  dststride = dstPicYuv->m_width;
>
> -    primitives.blockcpy_sp(width, height, dst, dststride, src, srcstride);
> +    int part = partitionFromSizes(width, height);
> +    primitives.luma_copy_ps[part](dst, dststride, src, srcstride);
>

queued, but this is not necessarily a performance improvement.  the
additional function overhead should be avoided by passing in the part enum
rather than width and height.  Most callers should already have the
partition enum.


>  }
>
>  void TComYuv::copyPartToPartChroma(TComYuv* dstPicYuv, uint32_t partIdx,
> uint32_t width, uint32_t height)
> _______________________________________________
> x265-devel mailing list
> x265-devel at videolan.org
> https://mailman.videolan.org/listinfo/x265-devel
>



-- 
Steve Borho
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20131112/6bcd142e/attachment.html>