As I posted earlier, there&#39;s a small error in the patch; x shouldn&#39;t be re-initted to zero after the first loop.<br><br>The x++ is correct, but it&#39;d be better style-wise to use x+=4 for w64 ... and x+=2 for the w32 loop.<br>
<br>Also note that upon fixing checkasm to use random data for the input, it appears that the assembly disagrees with the C, so any thoughts there would be appreciated.<br><br><div class="gmail_quote">On Fri, Mar 7, 2008 at 3:12 AM, Hannes Domani &lt;<a href="mailto:ssbssa@yahoo.de">ssbssa@yahoo.de</a>&gt; wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Hello<br>
<br>
when looking at the patch i was wondering if the<br>
following part is what you really intended (since x++<br>
is called twice):<br>
&gt; - &nbsp; &nbsp;if( qpel_idx &amp; 5 ) /* qpel interpolation needed<br>
<div class="Ih2E3d">*/<br>
&gt; + &nbsp; &nbsp;int x;<br>
&gt; + &nbsp; &nbsp;width = width &gt;&gt; 2;<br>
&gt; + &nbsp; &nbsp;for( x = 0; width - x &gt;= 2; x++ )<br>
&gt; &nbsp; &nbsp; &nbsp;{<br>
&gt; - &nbsp; &nbsp; &nbsp; &nbsp;uint8_t *src2 = src[hpel_ref1[qpel_idx]] +<br>
offset + ((mvx&amp;3) == 3);<br>
&gt; - &nbsp; &nbsp; &nbsp; &nbsp;x264_pixel_avg_wtab_mmxext[i_width&gt;&gt;2](<br>
&gt; - &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;dst, *i_dst_stride, src1,<br>
i_src_stride,<br>
&gt; - &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;src2, i_height );<br>
&gt; - &nbsp; &nbsp; &nbsp; &nbsp;return dst;<br>
&gt; + &nbsp; &nbsp; &nbsp; &nbsp;frame_init_lowres_core_sse2_w32(src_stride,<br>
dest_stride, height, width, src0, dst0, dsth, dstv,<br>
dstc );<br>
&gt; + &nbsp; &nbsp; &nbsp; &nbsp;src0 += 32;<br>
&gt; + &nbsp; &nbsp; &nbsp; &nbsp;dst0 += 16;<br>
&gt; + &nbsp; &nbsp; &nbsp; &nbsp;dsth += 16;<br>
&gt; + &nbsp; &nbsp; &nbsp; &nbsp;dstv += 16;<br>
&gt; + &nbsp; &nbsp; &nbsp; &nbsp;dstc += 16;<br>
&gt; + &nbsp; &nbsp; &nbsp; &nbsp;x++;<br>
&gt; &nbsp; &nbsp; &nbsp;}<br>
<br>
<br>
</div>regards<br>
Domani Hannes<br>
<br>
<br>
 &nbsp; &nbsp; &nbsp;Machen Sie Yahoo! zu Ihrer Startseite. Los geht&#39;s:<br>
<a href="http://de.yahoo.com/set" target="_blank">http://de.yahoo.com/set</a><br>
<div><div></div><div class="Wj3C7c"><br>
_______________________________________________<br>
x264-devel mailing list<br>
<a href="mailto:x264-devel@videolan.org">x264-devel@videolan.org</a><br>
<a href="http://mailman.videolan.org/listinfo/x264-devel" target="_blank">http://mailman.videolan.org/listinfo/x264-devel</a><br>
</div></div></blockquote></div><br>