commit 3e3b9342a20147ec6e4f89aa290e20277c1260ce
author Hui Su <huisu@google.com> Fri Apr 12 18:27:28 2019 +0000
committer Hui Su <huisu@google.com> Mon Apr 15 18:05:58 2019 +0000
tree 4585b58a64b0a1b9377127b595e10e77f170e3d5
parent a6fe432e4f6876755f7c0575ae834d5735519756

Revert "Revert "Use 64x64 SB size more aggressively""

This reverts commit f595b5c702a46d999c312b3bd82424e395682940.

Reason for revert: root cause of test failures has been fixed by 1c8d70b.

STATS_CHANGED

Change-Id: Ib57433a7570f4ff2dd64c4bdbcc0cad2eb349b51

av1/encoder/encoder.c

diff --git a/av1/encoder/encoder.c b/av1/encoder/encoder.c
index 3dc49f4..2f2eebc 100644
--- a/av1/encoder/encoder.c
+++ b/av1/encoder/encoder.c
@@ -330,15 +330,13 @@
 #endif
 
   // When superres / resize is on, 'cm->width / height' can change between
-  // calls, so we don't apply this heuristic there. Also, this heuristic gives
-  // compression gain for speed >= 2 only.
-  // Things break if superblock size changes per-frame which is why this
-  // heuristic is set based on configured speed rather than actual
-  // speed-features (which may change per-frame in future)
+  // calls, so we don't apply this heuristic there.
+  // Things break if superblock size changes between the first pass and second
+  // pass encoding, which is why this heuristic is not configured as a
+  // speed-feature.
   if (cpi->oxcf.superres_mode == SUPERRES_NONE &&
-      cpi->oxcf.resize_mode == RESIZE_NONE && cpi->oxcf.speed >= 2) {
-    return (cm->width >= 480 && cm->height >= 360) ? BLOCK_128X128
-                                                   : BLOCK_64X64;
+      cpi->oxcf.resize_mode == RESIZE_NONE && cpi->oxcf.speed >= 1) {
+    return AOMMIN(cm->width, cm->height) > 480 ? BLOCK_128X128 : BLOCK_64X64;
   }
 
   return BLOCK_128X128;