<div dir="ltr">Yes, that is (r6m) for coefficients we are loading from stack, is why that comments seems helpful to me at this stage. <div><br></div><div>Regards,</div><div>Praveen</div></div><div class="gmail_extra"><br><br>
<div class="gmail_quote">On Tue, Oct 8, 2013 at 11:44 AM, 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">
<div style="line-height:1.7;font-size:14px;font-family:arial"><div class="im"><div>+;-----------------------------------------------------------------------------<br>+; void filterHorizontal_p_p_4(pixel *src, intptr_t srcStride, pixel *dst, intptr_t dstStride, int width, int height, short const *coeff)<br>
</div>
</div><blockquote style="PADDING-LEFT:1ex;MARGIN:0px 0px 0px 0.8ex;BORDER-LEFT:#ccc 1px solid">
<div dir="ltr">
<div class="gmail_extra">
<div class="gmail_quote"><div class="im">
<div><br></div>
<div>I assume this comment is out of date?  you shouldn't need to pass width and height</div>
<div> </div>
</div><blockquote class="gmail_quote" style="PADDING-LEFT:1ex;MARGIN:0px 0px 0px 0.8ex;BORDER-LEFT:#ccc 1px solid">
<p></p><div class="im">+;-----------------------------------------------------------------------------<br>+INIT_XMM sse4<br>+cglobal filterHorizontal_p_p_4, 4, 5, 5, src, srcstride, dst, dststride<br></div>+    mov         r4,         r6m<br>
<p></p>
<p>They use r6m, so the comment is right, maybe need sync the code</p></blockquote></div></div></div></blockquote></div><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" target="_blank">https://mailman.videolan.org/listinfo/x265-devel</a><br>
<br></blockquote></div><br></div>