<div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><BLOCKQUOTE id="isReplyContent" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid">
<DIV dir="ltr">
<DIV class="gmail_extra"><BR>
<DIV class="gmail_quote">On Thu, Nov 14, 2013 at 12:07 AM, <SPAN dir="ltr"><<A href="mailto:murugan@multicorewareinc.com" target="_blank">murugan@multicorewareinc.com</A>></SPAN> wrote:<BR>
<BLOCKQUOTE class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid"># HG changeset patch<BR># User Murugan Vairavel <<A href="mailto:murugan@multicorewareinc.com">murugan@multicorewareinc.com</A>><BR># Date 1384409049 -19800<BR>#      Thu Nov 14 11:34:09 2013 +0530<BR># Node ID c9fdf510182348c979a58be67101446c84b36569<BR># Parent  481cdfc251de0f99ef0a3c4fd53c786b79b5f182<BR>asm: Unit test code for scale1D_128to64 Routine<BR><BR>diff -r 481cdfc251de -r c9fdf5101823 source/test/pixelharness.cpp<BR>--- a/source/test/pixelharness.cpp      Wed Nov 13 12:46:07 2013 +0530<BR>+++ b/source/test/pixelharness.cpp      Thu Nov 14 11:34:09 2013 +0530<BR>@@ -586,6 +586,29 @@<BR>     return true;<BR> }<BR><BR>+bool PixelHarness::check_scale_pixel_t(scale_t ref, scale_t opt)<BR>+{<BR>+    ALIGN_VAR_16(pixel, ref_dest[64 * 64]);<BR>+    ALIGN_VAR_16(pixel, opt_dest[64 * 64]);<BR>+<BR>+    memset(ref_dest, 0, sizeof(ref_dest));<BR>+    memset(opt_dest, 0, sizeof(opt_dest));<BR>+<BR>+    int j = 0;<BR>+    for (int i = 0; i < ITERS; i++)<BR>+    {<BR>+        opt(opt_dest, pbuf1 + j, STRIDE);<BR>+        ref(ref_dest, pbuf1 + j, STRIDE);<BR>+<BR>+        if (memcmp(ref_dest, opt_dest, 64 * 64 * sizeof(pixel)))<BR>+            return false;<BR>+<BR>+        j += INCR;<BR>+    }<BR>+<BR>+    return true;<BR>+}<BR>+<BR> bool PixelHarness::testPartition(int part, const EncoderPrimitives& ref, const EncoderPrimitives& opt)<BR> {<BR>     if (opt.satd[part])<BR>@@ -880,6 +903,15 @@<BR>             return false;<BR>         }<BR>     }<BR>+<BR>+    if (opt.scale1D_128to64)<BR>+    {<BR>+        if (!check_scale_pixel_t(ref.scale1D_128to64, opt.scale1D_128to64))<BR>+        {<BR>+            printf("scale1D_128to64 failed!\n");<BR>+            return false;<BR>+        }<BR>+    }<BR>     return true;<BR> }<BR><BR>@@ -1093,4 +1125,10 @@<BR>         printf("downscale");<BR>         REPORT_SPEEDUP(opt.frame_init_lowres_core, ref.frame_init_lowres_core, pbuf2, pbuf1, pbuf2, pbuf3, pbuf4, 64, 64, 64, 64);<BR>     }<BR>+<BR>+    if (opt.scale1D_128to64)<BR>+    {<BR>+        printf("scale1D_128to64");<BR>+        REPORT_SPEEDUP(opt.scale1D_128to64, ref.scale1D_128to64, pbuf2, pbuf1, 64);<BR>+    }<BR> }<BR>diff -r 481cdfc251de -r c9fdf5101823 source/test/pixelharness.h<BR>--- a/source/test/pixelharness.h        Wed Nov 13 12:46:07 2013 +0530<BR>+++ b/source/test/pixelharness.h        Thu Nov 14 11:34:09 2013 +0530<BR>@@ -61,6 +61,8 @@<BR>     bool check_blockfill_s(blockfill_s_t ref, blockfill_s_t opt);<BR><BR>     bool check_pixel_sub_ps(pixel_sub_ps_t ref, pixel_sub_ps_t opt);<BR>+<BR>+    bool check_scale_pixel_t(scale_t ref, scale_t opt);<BR></BLOCKQUOTE>
<DIV><BR></DIV>
<DIV>drop _t suffix from the function name</DIV>
<DIV> </DIV>
<DIV>increment _pp in end of name is better, since this function convert Pixel to Pixel</DIV>
<DIV> </DIV></DIV></DIV></DIV></BLOCKQUOTE></div>