commit 2e2e4110ff1c31651355bb434b10dbef89e79976
author chiyotsai <chiyotsai@google.com> Mon May 11 12:37:21 2020 -0700
committer Chi Yo Tsai <chiyotsai@google.com> Thu May 21 21:41:17 2020 +0000
tree 9d0673d116651dfb1b681b46ef5d9e779242a550
parent 154af70e7cecd0160988e43a3f0943729dec4a45

Fix a ref_frame's rate being double-counted in handle_intra_mode

Performance is more or less neutral.

STATS_CHANGED

Change-Id: I709f82633632b8b1e7428c0896219ff1084bf204

av1/encoder/intra_mode_search.c

diff --git a/av1/encoder/intra_mode_search.c b/av1/encoder/intra_mode_search.c
index d4f413f..c99841c 100644
--- a/av1/encoder/intra_mode_search.c
+++ b/av1/encoder/intra_mode_search.c
@@ -1765,9 +1765,6 @@
   const int skip_ctx = av1_get_skip_txfm_context(xd);
 
   int known_rate = mode_cost;
-  // TODO(chiyotsai@google.com): known rate seems to be counting ref_frame_cost
-  // twice here. Try to remove this.
-  known_rate += ref_frame_cost;
   if (mode != DC_PRED && mode != PAETH_PRED) known_rate += intra_cost_penalty;
   known_rate += AOMMIN(mode_costs->skip_txfm_cost[skip_ctx][0],
                        mode_costs->skip_txfm_cost[skip_ctx][1]);