commit 4d8c6fc2e60ff47dc84776c2b8d4ac1403e5fb15
author Debargha Mukherjee <debargha@google.com> Thu Dec 07 11:25:18 2017 -0800
committer Debargha Mukherjee <debargha@google.com> Sat Dec 09 03:00:50 2017 +0000
tree e122357886e54f4122c679888179c091369ded4e
parent c505fe3aaaabeb2101e92960582d134821086b99

Some rd fixes related to skip cost

Change-Id: Ia5b5685fe22293767d0e1d5c1747a33ba9d79e0a

av1/encoder/rdopt.c

diff --git a/av1/encoder/rdopt.c b/av1/encoder/rdopt.c
index 1859e2d..379b284 100644
--- a/av1/encoder/rdopt.c
+++ b/av1/encoder/rdopt.c
@@ -4440,15 +4440,19 @@
           return 0;
         }
         av1_merge_rd_stats(rd_stats, &pn_rd_stats);
-        this_rd += AOMMIN(RDCOST(x->rdmult, pn_rd_stats.rate, pn_rd_stats.dist),
-                          RDCOST(x->rdmult, 0, pn_rd_stats.sse));
+        this_rd +=
+            AOMMIN(RDCOST(x->rdmult, pn_rd_stats.rate, pn_rd_stats.dist),
+                   RDCOST(x->rdmult, pn_rd_stats.zero_rate, pn_rd_stats.sse));
         block += step;
       }
     }
   }
-
-  this_rd = AOMMIN(RDCOST(x->rdmult, rd_stats->rate, rd_stats->dist),
-                   RDCOST(x->rdmult, 0, rd_stats->sse));
+  int64_t zero_rd = RDCOST(x->rdmult, rd_stats->zero_rate, rd_stats->sse);
+  this_rd = RDCOST(x->rdmult, rd_stats->rate, rd_stats->dist);
+  if (zero_rd < this_rd) {
+    this_rd = zero_rd;
+    rd_stats->skip = 1;
+  }
   if (this_rd > ref_best_rd) is_cost_valid = 0;
 
   if (!is_cost_valid) {
@@ -5025,7 +5029,7 @@
   MB_MODE_INFO *const mbmi = &xd->mi[0]->mbmi;
   int plane;
   int is_cost_valid = 1;
-  int64_t this_rd;
+  int64_t this_rd = 0;
 
   if (ref_best_rd < 0) is_cost_valid = 0;
 
@@ -5035,12 +5039,6 @@
   const BLOCK_SIZE bsizec = scale_chroma_bsize(
       bsize, xd->plane[1].subsampling_x, xd->plane[1].subsampling_y);
 
-#if 0   // CONFIG_EXT_TX
-  if (is_rect_tx(mbmi->tx_size)) {
-    return super_block_uvrd(cpi, x, rd_stats, bsize, ref_best_rd);
-  }
-#endif  // CONFIG_EXT_TX
-
   if (is_inter_block(mbmi) && is_cost_valid) {
     for (plane = 1; plane < MAX_MB_PLANE; ++plane)
       av1_subtract_plane(x, bsizec, plane);
@@ -5094,7 +5092,7 @@
       av1_merge_rd_stats(rd_stats, &pn_rd_stats);
 
       this_rd = AOMMIN(RDCOST(x->rdmult, rd_stats->rate, rd_stats->dist),
-                       RDCOST(x->rdmult, 0, rd_stats->sse));
+                       RDCOST(x->rdmult, rd_stats->zero_rate, rd_stats->sse));
 
       if (this_rd > ref_best_rd) {
         is_cost_valid = 0;