shithub: libvpx

Download patch

ref: bcbc6ed82df08ea7b02b5a7b996ad4cbb80e1293
parent: 9311ef18b4b4eff0da3adf9d702a34f489a270ff
author: Marco Paniconi <marpan@google.com>
date: Fri Oct 6 18:14:56 EDT 2017

Revert "Speed >=5 real-time: add TM intra mode for high_source_sad."

This reverts commit 9311ef18b4b4eff0da3adf9d702a34f489a270ff.

Reason for revert:
Notice small regression in some clips.
Will revisit in another change.

Original change's description:
> Speed >=5 real-time: add TM intra mode for high_source_sad.
> 
> Small/neutral change in metrics or speed for ytlive.
> Some improvement in quality on frames with big content change.
> 
> Change-Id: Ib3b0703a5f28ea6710e90324436e27598ab7384d

TBR=marpan@google.com,builds@webmproject.org,jianj@google.com

Change-Id: I9d8ec5195bb05ddf329d325699355185affb9b13
No-Presubmit: true
No-Tree-Checks: true
No-Try: true

--- a/vp9/encoder/vp9_speed_features.c
+++ b/vp9/encoder/vp9_speed_features.c
@@ -507,16 +507,12 @@
         for (i = 0; i < BLOCK_SIZES; ++i)
           sf->intra_y_mode_bsize_mask[i] = INTRA_DC_TM_H_V;
       } else {
-        for (i = 0; i < BLOCK_SIZES; ++i) {
-          if (i > BLOCK_16X16) {
+        for (i = 0; i < BLOCK_SIZES; ++i)
+          if (i > BLOCK_16X16)
             sf->intra_y_mode_bsize_mask[i] = INTRA_DC;
-          } else {
-            if (cpi->rc.high_source_sad)
-              sf->intra_y_mode_bsize_mask[i] = INTRA_DC_TM_H_V;
-            else
-              sf->intra_y_mode_bsize_mask[i] = INTRA_DC_H_V;
-          }
-        }
+          else
+            // Use H and V intra mode for block sizes <= 16X16.
+            sf->intra_y_mode_bsize_mask[i] = INTRA_DC_H_V;
       }
     }
     if (content == VP9E_CONTENT_SCREEN) {