<div dir="ltr">Min,<div>Can you please share the impact that this sequence of 3 patches has on performance for our presets. What machine did you do the test on?</div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">Pradeep Ramachandran, PhD<div>Solution Architect,</div><div><a href="http://pradeeprama.info/" target="_blank">pradeeprama.info/</a></div><div><a href="http://www.multicorewareinc.com/" target="_blank">www.multicorewareinc.com/</a></div><div>Ph: +91 99627 82018</div></div></div></div></div></div></div></div></div>
<br><div class="gmail_quote">On Tue, Dec 22, 2015 at 7:19 AM, Min Chen <span dir="ltr"><<a href="mailto:chenm003@163.com" target="_blank">chenm003@163.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"># HG changeset patch<br>
# User Min Chen <<a href="mailto:chenm003@163.com">chenm003@163.com</a>><br>
# Date 1450728715 21600<br>
# Node ID 3e9e45fd692d79806e53d69c2ed7a20b3e24e671<br>
# Parent d8c3eded1440670bde63e2fb5bec0e80ff6e2d67<br>
reduce pointer operators on I400<br>
---<br>
source/encoder/framefilter.cpp | 7 ++++---<br>
1 files changed, 4 insertions(+), 3 deletions(-)<br>
<br>
diff -r d8c3eded1440 -r 3e9e45fd692d source/encoder/framefilter.cpp<br>
--- a/source/encoder/framefilter.cpp Mon Dec 21 13:46:28 2015 -0600<br>
+++ b/source/encoder/framefilter.cpp Mon Dec 21 14:11:55 2015 -0600<br>
@@ -507,7 +507,7 @@<br>
const intptr_t stride = reconPic->m_stride;<br>
const intptr_t strideC = reconPic->m_strideC;<br>
pixel *pixY = reconPic->getLumaAddr(lineStartCUAddr);<br>
- // // MUST BE check I400 since m_picOrg uninitialize in that case<br>
+ // MUST BE check I400 since m_picOrg uninitialize in that case<br>
pixel *pixU = (m_param->internalCsp != X265_CSP_I400) ? reconPic->getCbAddr(lineStartCUAddr) : NULL;<br>
pixel *pixV = (m_param->internalCsp != X265_CSP_I400) ? reconPic->getCrAddr(lineStartCUAddr) : NULL;<br>
int copySizeY = realW;<br>
@@ -560,13 +560,14 @@<br>
if (row == FrameFilter::ParallelFilter::numRows - 1)<br>
{<br>
pixY += (realH - 1) * stride;<br>
- pixU += ((realH >> vChromaShift) - 1) * strideC;<br>
- pixV += ((realH >> vChromaShift) - 1) * strideC;<br>
for (uint32_t y = 0; y < lumaMarginY; y++)<br>
memcpy(pixY + (y + 1) * stride, pixY, copySizeY * sizeof(pixel));<br>
<br>
if (m_param->internalCsp != X265_CSP_I400)<br>
{<br>
+ pixU += ((realH >> vChromaShift) - 1) * strideC;<br>
+ pixV += ((realH >> vChromaShift) - 1) * strideC;<br>
+<br>
for (uint32_t y = 0; y < chromaMarginY; y++)<br>
{<br>
memcpy(pixU + (y + 1) * strideC, pixU, copySizeC * sizeof(pixel));<br>
<br>
_______________________________________________<br>
x265-devel mailing list<br>
<a href="mailto:x265-devel@videolan.org">x265-devel@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/x265-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/x265-devel</a><br>
</blockquote></div><br></div>