]> granicus.if.org Git - libvpx/commitdiff
Merge "Enable encoder to avoid 8x4 or 4x8 partitions"
authorYaowu Xu <yaowu@google.com>
Thu, 7 Jan 2016 23:00:00 +0000 (23:00 +0000)
committerGerrit Code Review <noreply-gerritcodereview@google.com>
Thu, 7 Jan 2016 23:00:00 +0000 (23:00 +0000)
vp9/encoder/vp9_pickmode.c
vp9/encoder/vp9_rdopt.c

index 0d1056075fa2c436296e2ef8d9ab41a20d5c8596..71b8bdba02146454914244140411d79cc5d262c1 100644 (file)
@@ -1858,6 +1858,13 @@ void vp9_pick_inter_mode_sub8x8(VP9_COMP *cpi, MACROBLOCK *x,
     if (ref_frame_skip_mask & (1 << ref_frame))
       continue;
 
+#if CONFIG_BETTER_HW_COMPATIBILITY
+    if ((bsize == BLOCK_8X4 || bsize == BLOCK_4X8) &&
+        ref_frame > INTRA_FRAME &&
+        vp9_is_scaled(&cm->frame_refs[ref_frame - 1].sf))
+      continue;
+#endif
+
     // TODO(jingning, agrange): Scaling reference frame not supported for
     // sub8x8 blocks. Is this supported now?
     if (ref_frame > INTRA_FRAME &&
index 9402a4a2b7a3810d8b20ce55e7bc21d4a5c1d6d9..b8d17205ddc1cc732820c998864d08489b231443 100644 (file)
@@ -3818,6 +3818,16 @@ void vp9_rd_pick_inter_mode_sub8x8(VP9_COMP *cpi,
     ref_frame = vp9_ref_order[ref_index].ref_frame[0];
     second_ref_frame = vp9_ref_order[ref_index].ref_frame[1];
 
+#if CONFIG_BETTER_HW_COMPATIBILITY
+    // forbid 8X4 and 4X8 partitions if any reference frame is scaled.
+    if (bsize == BLOCK_8X4 || bsize == BLOCK_4X8) {
+      int ref_scaled = vp9_is_scaled(&cm->frame_refs[ref_frame - 1].sf);
+      if (second_ref_frame > INTRA_FRAME)
+        ref_scaled += vp9_is_scaled(&cm->frame_refs[second_ref_frame - 1].sf);
+      if (ref_scaled)
+        continue;
+    }
+#endif
     // Look at the reference frame of the best mode so far and set the
     // skip mask to look at a subset of the remaining modes.
     if (ref_index > 2 && sf->mode_skip_start < MAX_MODES) {