commit 2c50f9ae413f2035ab618f23d7fbdf7d0e53da08
author Debargha Mukherjee <debargha@google.com> Wed Nov 15 08:04:57 2017 -0800
committer Debargha Mukherjee <debargha@google.com> Mon Nov 20 17:06:10 2017 +0000
tree 0b74f23e68c080531ef6bd86b4780765b8aa941d
parent a496c673dbffacfc0d98230a820fe6a7680e1dc6

Some RD fixes

Change-Id: Ie6b41ac068846da3c87bdcaa94180777edb27d1f

av1/encoder/rd.h

diff --git a/av1/encoder/rd.h b/av1/encoder/rd.h
index 8a0ac76..f50331f 100644
--- a/av1/encoder/rd.h
+++ b/av1/encoder/rd.h
@@ -387,6 +387,7 @@
   rd_stats_dst->rate += rd_stats_src->rate;
   rd_stats_dst->dist += rd_stats_src->dist;
   rd_stats_dst->sse += rd_stats_src->sse;
+  rd_stats_dst->zero_rate += rd_stats_src->zero_rate;
   rd_stats_dst->skip &= rd_stats_src->skip;
   rd_stats_dst->invalid_rate &= rd_stats_src->invalid_rate;
 #if CONFIG_RD_DEBUG

av1/encoder/rdopt.c

diff --git a/av1/encoder/rdopt.c b/av1/encoder/rdopt.c
index a381098..8b4e23d 100644
--- a/av1/encoder/rdopt.c
+++ b/av1/encoder/rdopt.c
@@ -4183,16 +4183,16 @@
           return;
         }
         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;
         if (rd_info_tree != NULL) rd_info_tree += 1;
       }
     }
   }
-
   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;
 
   if (!is_cost_valid) {