shithub: libvpx

Download patch

ref: 23a64c890e99e7c70365f12694c76e8605a11f85
parent: 66180f228c9b369f920013f93c64b9bd54d88e69
author: Ronald S. Bultje <rbultje@google.com>
date: Mon Oct 15 09:54:05 EDT 2012 

Remove test from B_MODE_INFO.

It is essentially a duplicate of mode for RD-only purposes. Removing it
saves us 4 bytes per B_MODE_INFO, or ~0.5MB for a 1080p video encode.

Change-Id: I0a54db5f51658b3946d7efb1ca6e8cfbda0cdf88


--- a/vp8/common/blockd.h
+++ b/vp8/common/blockd.h
@@ -237,7 +237,6 @@
   struct {
     B_PREDICTION_MODE first;
 #if CONFIG_HYBRIDTRANSFORM8X8 || CONFIG_HYBRIDTRANSFORM || CONFIG_HYBRIDTRANSFORM16X16
-    B_PREDICTION_MODE test;
     TX_TYPE           tx_type;
 #endif
 
--- a/vp8/encoder/encodeintra.c
+++ b/vp8/encoder/encodeintra.c
@@ -74,7 +74,6 @@
 
 #if CONFIG_HYBRIDTRANSFORM
   if (x->q_index < ACTIVE_HT) {
-    b->bmi.as_mode.test = b->bmi.as_mode.first;
     txfm_map(b, b->bmi.as_mode.first);
     vp8_fht_c(be->src_diff, be->coeff, 32, b->bmi.as_mode.tx_type, 4);
     vp8_ht_quantize_b_4x4(be, b);
--- a/vp8/encoder/rdopt.c
+++ b/vp8/encoder/rdopt.c
@@ -1182,6 +1182,7 @@
       // Do we need to do this for mode2 also?
       if (mode == B_LD_PRED || mode == B_VL_PRED)
         continue;
+      b->bmi.as_mode.first = mode;
       rate = bmode_costs[mode];
 
 #if CONFIG_COMP_INTRA_PRED
@@ -1200,7 +1201,6 @@
 
 #if CONFIG_HYBRIDTRANSFORM
       if (active_ht) {
-        b->bmi.as_mode.test = mode;
         txfm_map(b, mode);
         vp8_fht_c(be->src_diff, be->coeff, 32, b->bmi.as_mode.tx_type, 4);
         vp8_ht_quantize_b_4x4(be, b);
-- 
⑨