commit d80d63f477a8a20a72365a20fee1acb55b50cd7c
author Debargha Mukherjee <debargha@google.com> Wed May 17 15:53:27 2017 -0700
committer Debargha Mukherjee <debargha@google.com> Thu May 18 11:33:27 2017 -0700
tree 72ded87427451f623b30b2f180748d15c5a70c91
parent 8517d6aea1935cee4e496a3142fb23ad3790d186

Fixes a mv pred issue introduced in a prior patch

Fixes a baseline BDRATE drop issue introduced in
0743857c1a01d40a369c89af27bb1c819253e0d9.

Partially reverts the above patch.

BUG=aomedia:544

Change-Id: Ia7cd89330cab189d22b58e183951fc6429021f42

av1/encoder/rd.c

diff --git a/av1/encoder/rd.c b/av1/encoder/rd.c
index 3ac9353..ff2532d 100644
--- a/av1/encoder/rd.c
+++ b/av1/encoder/rd.c
@@ -742,24 +742,19 @@
   int max_mv = 0;
   uint8_t *src_y_ptr = x->plane[0].src.buf;
   uint8_t *ref_y_ptr;
-  int num_mv_refs = 0;
   MV pred_mv[MAX_MV_REF_CANDIDATES + 1];
-  if (cpi->sf.adaptive_motion_search && block_size < x->max_partition_size) {
-    pred_mv[num_mv_refs] = x->pred_mv[ref_frame];
-    num_mv_refs++;
+  int num_mv_refs = 0;
+
+  pred_mv[num_mv_refs++] = x->mbmi_ext->ref_mvs[ref_frame][0].as_mv;
+  if (x->mbmi_ext->ref_mvs[ref_frame][0].as_int !=
+      x->mbmi_ext->ref_mvs[ref_frame][1].as_int) {
+    pred_mv[num_mv_refs++] = x->mbmi_ext->ref_mvs[ref_frame][1].as_mv;
   }
-  if (x->mbmi_ext->ref_mv_count[ref_frame] > 0) {
-    pred_mv[num_mv_refs] = x->mbmi_ext->ref_mvs[ref_frame][0].as_mv;
-    num_mv_refs++;
-  }
-  if (x->mbmi_ext->ref_mv_count[ref_frame] > 1) {
-    if (x->mbmi_ext->ref_mvs[ref_frame][0].as_int !=
-        x->mbmi_ext->ref_mvs[ref_frame][1].as_int) {
-      pred_mv[num_mv_refs] = x->mbmi_ext->ref_mvs[ref_frame][1].as_mv;
-      num_mv_refs++;
-    }
-  }
+  if (cpi->sf.adaptive_motion_search && block_size < x->max_partition_size)
+    pred_mv[num_mv_refs++] = x->pred_mv[ref_frame];
+
   assert(num_mv_refs <= (int)(sizeof(pred_mv) / sizeof(pred_mv[0])));
+
   // Get the sad for each candidate reference mv.
   for (i = 0; i < num_mv_refs; ++i) {
     const MV *this_mv = &pred_mv[i];