shithub: libvpx

Info  •  Files  •  Log  •  Branches

Refactor column integral projection computation

Move the scaling factor outside column projection. This avoids
repeated calculation of the same scaling factor. Profiling shows
that the percentage of vp9_int_pro_col_sse2 of overall cycles
goes from 2.29% down to 1.88%.

Change-Id: I5ac4e324ab2d7f33ba2de66dd2a12e04e04dfd66

--- a/vp9/encoder/vp9_mcomp.c

+++ b/vp9/encoder/vp9_mcomp.c

@@ -1805,6 +1805,7 @@

   MV *tmp_mv = &xd->mi[0].src_mi->mbmi.mv[0].as_mv;

   unsigned int best_sad, tmp_sad, this_sad[4];

   MV this_mv;

+  const int norm_factor = 3 + (bw >> 5);

 #if CONFIG_VP9_HIGHBITDEPTH

   tmp_mv->row = 0;

@@ -1822,7 +1823,7 @@

   ref_buf = xd->plane[0].pre[0].buf - (bh >> 1) * ref_stride;

   for (idx = 0; idx < search_height; ++idx) {

-    vbuf[idx] = vp9_int_pro_col(ref_buf, bw);

+    vbuf[idx] = vp9_int_pro_col(ref_buf, bw) >> norm_factor;

     ref_buf += ref_stride;

   }

@@ -1834,7 +1835,7 @@

   src_buf = x->plane[0].src.buf;

   for (idx = 0; idx < bh; ++idx) {

-    src_vbuf[idx] = vp9_int_pro_col(src_buf, bw);

+    src_vbuf[idx] = vp9_int_pro_col(src_buf, bw) >> norm_factor;

     src_buf += src_stride;

   }

--- a/vp9/encoder/x86/vp9_avg_intrin_sse2.c

+++ b/vp9/encoder/x86/vp9_avg_intrin_sse2.c

@@ -112,7 +112,6 @@

   __m128i s0 = _mm_sad_epu8(src_line, zero);

   __m128i s1;

   int i;

-  const int norm_factor = 3 + (width >> 5);

   for (i = 16; i < width; i += 16) {

     ref += 16;

@@ -124,7 +123,7 @@

   s1 = _mm_srli_si128(s0, 8);

   s0 = _mm_adds_epu16(s0, s1);

-  return _mm_extract_epi16(s0, 0) >> norm_factor;

+  return _mm_extract_epi16(s0, 0);

 }

 int vp9_vector_var_sse2(int16_t const *ref, int16_t const *src,