commit b4fb30469499c7865811d279657a48417035ad1e
author Deepa K G <deepa.kg@ittiam.com> Wed Feb 03 13:31:40 2021 +0530
committer Yunqing Wang <yunqingwang@google.com> Wed Feb 10 19:10:42 2021 +0000
tree a52c6f5ff2ea641bb4fbd2ba50914fec13adce54
parent c744ef77d257a366505f11d47e92822198e18298

Remove redundant initialization of costs

Avoid repeated calling of the functions
av1_fill_coeff_costs(), av1_fill_mode_rates()
and av1_fill_mv_costs() during encoding when
cost update frequency is set as COST_UPD_TILE.

Change-Id: I08b5efd519f535f23e3da6b75460e7fba4c9aa2f

av1/encoder/encodeframe_utils.c

diff --git a/av1/encoder/encodeframe_utils.c b/av1/encoder/encodeframe_utils.c
index b832d91..6251233 100644
--- a/av1/encoder/encodeframe_utils.c
+++ b/av1/encoder/encodeframe_utils.c
@@ -1333,10 +1333,9 @@
   MACROBLOCKD *const xd = &x->e_mbd;
 
   switch (cpi->oxcf.cost_upd_freq.coeff) {
-    case COST_UPD_OFF: break;
+    case COST_UPD_OFF:
     case COST_UPD_TILE:  // Tile level
-      if (mi_row != tile_info->mi_row_start) break;
-      AOM_FALLTHROUGH_INTENDED;
+      break;
     case COST_UPD_SBROW:  // SB row level in tile
       if (mi_col != tile_info->mi_col_start) break;
       AOM_FALLTHROUGH_INTENDED;
@@ -1347,10 +1346,9 @@
   }
 
   switch (cpi->oxcf.cost_upd_freq.mode) {
-    case COST_UPD_OFF: break;
+    case COST_UPD_OFF:
     case COST_UPD_TILE:  // Tile level
-      if (mi_row != tile_info->mi_row_start) break;
-      AOM_FALLTHROUGH_INTENDED;
+      break;
     case COST_UPD_SBROW:  // SB row level in tile
       if (mi_col != tile_info->mi_col_start) break;
       AOM_FALLTHROUGH_INTENDED;
@@ -1360,10 +1358,9 @@
     default: assert(0);
   }
   switch (cpi->oxcf.cost_upd_freq.mv) {
-    case COST_UPD_OFF: break;
+    case COST_UPD_OFF:
     case COST_UPD_TILE:  // Tile level
-      if (mi_row != tile_info->mi_row_start) break;
-      AOM_FALLTHROUGH_INTENDED;
+      break;
     case COST_UPD_SBROW:  // SB row level in tile
       if (mi_col != tile_info->mi_col_start) break;
       AOM_FALLTHROUGH_INTENDED;