commit 5027edce83ea95e2b9350732933b07e4e5a1633f
author Hui Su <huisu@google.com> Thu Apr 02 11:45:24 2020 -0700
committer Hui Su <huisu@google.com> Fri Apr 03 04:30:51 2020 +0000
tree 0901bcf30995aed3ce2d101e89bd44aad2798604
parent 39d036229662d6a6506b609f5da3c39cdfc9328c

Code simplification in select_tx_block()

Change-Id: I708ada2b09715210fec3d3667cfa8496d77cdb6f

av1/encoder/tx_search.c

diff --git a/av1/encoder/tx_search.c b/av1/encoder/tx_search.c
index d0cfcd4..5e678f9 100644
--- a/av1/encoder/tx_search.c
+++ b/av1/encoder/tx_search.c
@@ -2569,11 +2569,8 @@
   }
 
   MACROBLOCKD *const xd = &x->e_mbd;
-  const int max_blocks_high = max_block_high(xd, plane_bsize, 0);
-  const int max_blocks_wide = max_block_wide(xd, plane_bsize, 0);
-  if (blk_row >= max_blocks_high || blk_col >= max_blocks_wide) return;
-
-  const int bw = mi_size_wide[plane_bsize];
+  assert(blk_row < max_block_high(xd, plane_bsize, 0) &&
+         blk_col < max_block_wide(xd, plane_bsize, 0));
   MB_MODE_INFO *const mbmi = xd->mi[0];
   const int ctx = txfm_partition_context(tx_above + blk_col, tx_left + blk_row,
                                          mbmi->sb_type, tx_size);
@@ -2648,6 +2645,7 @@
     }
     mbmi->tx_size = tx_size_selected;
     update_txk_array(xd, blk_row, blk_col, tx_size, no_split.tx_type);
+    const int bw = mi_size_wide[plane_bsize];
     set_blk_skip(x, 0, blk_row * bw + blk_col, rd_stats->skip);
   } else {
     *rd_stats = split_rd_stats;
@@ -3131,8 +3129,8 @@
   int block = 0;
 
   av1_init_rd_stats(rd_stats);
-  for (int idy = 0; idy < mi_height; idy += bh) {
-    for (int idx = 0; idx < mi_width; idx += bw) {
+  for (int idy = 0; idy < max_block_high(xd, bsize, 0); idy += bh) {
+    for (int idx = 0; idx < max_block_wide(xd, bsize, 0); idx += bw) {
       const int64_t best_rd_sofar =
           (rd_thresh == INT64_MAX)
               ? INT64_MAX