Change in rddiv parameter to make it a power of 2

Converts the constant rddiv parameter to 128 (from 100) and
implements RDCOST with bit-shift rather than multiplication.
Other parameters are also adjusted to roughly keep the same
balance between Rate and Distortion.

There is a slight speed-up of about 0.5-1% (at speed 0) as
testted on football_cif.

There is a slight change in performance due to small change
in the parameters.
derfraw300: +0.033%
stdhdraw250; +0.102%

Change-Id: I70ac69f58fa71c83108f68fe41796cd19d1fc760

Merge "Adjustment to mv cost parameters"

Merge "Fix intra dist model of skip_encode feature"

Merge "Allow sub8x8 intra modes test for alt frame coding"

Merge "Disable MODE_TEST_HIT_STATS"

Adjustment to mv cost parameters

Increases these parameters.
There is a small efficiency gain.

Change-Id: Ie5f0ddb39c907d335e0dafa5eb112365a81f4542
derfraw300: +0.091%
stdhdraw250: +0.238%

Merge "Simplifying and inlining k_cvtlo_epi16 and k_cvthi_epi16"

Fix intra dist model of skip_encode feature

The intra mode distortion adjustment for skip_encode feature was
broken in the refactoring cc91851. This commit fixes it and tunes
the distortion models used therein.

Change-Id: I0d676e82f8e855536a90cf9b3e3fdefafcd886c6

Merge "Added #define of snprintf for MSVC"

Added #define of snprintf for MSVC

snprintf is not supported by MSVC, the commit replace it with the msvc
variant _snprintf to enable build.

Change-Id: I686943a78c289bae6b486a5e75effad5f86c24de

Merge "Clean-ups in rdopt.c"

Clean-ups in rdopt.c

Some minor cleanups in preparation for experimentation with
some encode parameters and thresholds

Change-Id: I449d66da97eae0a7acdf4aae374e2f9111342056

Merge "Deprecate the use of PARTITION_INFO from encoder"

Deprecate the use of PARTITION_INFO from encoder

Use b_mode_info to store the inter prediction mode of sub8x8 block,
in replacement of the use of partition_info. Remove redundant buffer
update for partition_info. For bus_cif at 2000 kbps, this seem to make
speed 0 about 1% faster.

Change-Id: Id1b3be45e75a24fb4b42335ac480c23e440978f6

mips dsp-ase r2 vp9 decoder bilinear convolve optimizations

Change-Id: Ic31b4ef85e65070b4f8b9f26e068ccfaae00c4f0

Merge "vp9_blockd.h: update get_tx_eob() signature"

Remove extra line in decode_coefs

Change-Id: Id1fde9920d60c6991a8ef6de5103ae3e578312ed

Merge "All zero coeff skip in IDCT 32x32"

All zero coeff skip in IDCT 32x32

When all coefficients are zeros, skip the corresponding 1-D inverse
transform. This practice has been used in the SSE2 implementation of
inverse 32x32 DCT. This commit imports this algorithm into the C code.

Change-Id: I0f58bfcb183a569fab85d524d5d9cf8ae8653f86

Merge "Removing redundant vp9_pt_energy_class declarations."

Merge "easy to fix cpplint issue in rdopt.c"

Merge "Fix MSVC warning."

Change to allow less rectangular partion check

For CpuUsed 1 & 2, this commit allow to skip retangular partition check
when NONE is better than SPLIT. It also changed to allow such logic
on alt ref frame coding rather than use square partition all them. The
change has gain compressio about .3% on yt and ythd for both 1&2, It
helped .6% compression on cif and stdhd for both CpuUsed 1&2.

Change-Id: I814b653baf89f59acd20e042629a12938a1bd4e5

Merge "Rate control parameter adjustment"

easy to fix cpplint issue in rdopt.c

Change-Id: Id093816146de0d100f0c6ae2542aaa427dbab2d8

Merge "cpplint issue with convolve resolved"

cpplint issue with convolve resolved

Change-Id: I38b2100f1a64cb067c63f4e1662c36914b3569df

cpplint issue missed in first pass for vp9_bitstream.c

Change-Id: Ia725748acbc2a3f825f0d208f26522a0412301fa

Merge "cpplint vp9_variance_sse2.c"

Rate control parameter adjustment

Adjusts the bits per mb parameter about 10% smaller.

Results at speed 0:
fullderfraw: +0.255%
fullstdhdraw: +0.262%
stdhdraw250: +0.291%

Change-Id: I2b7317ac3f61737bc77eb5470aad870cade83fa5

Merge "cpplint errors in vp9_onyx_if.h"

Merge "Giving consistent names to IDCT 8x8 functions."

Removing redundant vp9_pt_energy_class declarations.

Declaring vp9_pt_energy_class in vp9_entropy.h instead of many external
places.

Change-Id: I66e8a3fc119a43f88d130d0dae4133c825a047a3

cpplint errors in vp9_onyx_if.h

Slightly bigger change -> broke up encode_frame_to_datarate,  lots
of line length fixes.

Change-Id: I7c53325e954de130f3fe1a6656626efc6705be82

Using inter_mode_offset_function instead of duplicated code.

Change-Id: I8de865cd1deca07b5c92c225782f0867367e9a11

Merge "Cleaning up foreach_predicted_block_in_plane() function."

Merge "cpplint issues resolved vp9_ratectrl.c"

cpplint issues resolved vp9_ratectrl.c

Change-Id: Iae7674b0c946a5ac01617840b3f62965c654d920

Merge "cpplint problems resolved with vp9_firstpass.c"

Merge "cpplint issues resolved in vp9_mcomp.c"

Disable MODE_TEST_HIT_STATS

This flag is for stats generation and testing and should not
be checked in as enabled by default.

Change-Id: I4ea57dbcf49790f14777f598ddd3dc37dcc7a6bb

vp9_blockd.h: update get_tx_eob() signature

as the name implies, the segmentation pointer can be const

Change-Id: I945f01a077c112ec86c00e35a1e9395bc230c2d9

Fix MSVC warning.

A new set of MSVC warnings were introduced by change
I3f36d3f7cd8d15195a6e2fafd1777cdaf9ecb847

In particular MSVC does not like:-

typedef const int16_t subpel_kernel[SUBPEL_TAPS];

struct subpix_fn_table {
  const subpel_kernel *filter_x;
  const subpel_kernel *filter_y;
};

causes  new warning in MSVC.
warning C4114: same type qualifier used more than once

Change-Id: Iae596fd13aadf36169faf00c68eabe9a32a9b156

Merge changes I8a106dd6,Iec442603

* changes:
  d153 intra prediction (16x16) ssse3 using bytes
  d153 intra prediction ssse3 using bytes

Giving consistent names to IDCT 8x8 functions.

Renames:
  vp9_short_idct8x8_add    -> vp9_idct8x8_64_add
  vp9_short_idct8x8_1_add  -> vp9_idct8x8_1_add
  vp9_short_idct8x8_10_add -> vp9_idct8x8_10_add
  vp9_idct_add_8x8         -> vp9_idct8x8_add

Change-Id: Ifb8d3a45b4c0397aa805b30463f3d14581bf72c1

Merge "Adding assign_mv() function to reduce code duplication."

Merge "Giving consistent names to IDCT/IWHT functions."

Merge "encodemb cpplint issues revisited."

NOLINT issue with headers that's hard to avoid do to config.h issue

Change-Id: Ibd0b3414cdea05bc2fd6d0aa35808e44b3db8d96

remaining cpplint issue in vp9_decode_frame

Change-Id: Ia3030882c5276dc1f8e6b6c82b9eb301f00b6bbc

encodemb cpplint issues revisited.

Change-Id: Id5f25b74e2207bf44b6f6c8ffe548fa30fd78b4d

cpplint problems resolved with vp9_firstpass.c

Change-Id: Ic7b7014a0d857585bfd4baaea1d5c27ffe355642

Merge "unused typedef in vp9_variance.h"

Merge "cpplint issues with vp9_boolhuff.c resolved"

Merge "added nolint to function that doesn't seem easy to breakup"

Merge "cpplint issues in vp9_lookahead.c"

cpplint issues resolved in vp9_mcomp.c

Change-Id: I2c2f83f4dfa2782fc6b0aa6db3ba2c4e6e423ffa

Merge changes Idbfabe42,I788f1a30

* changes:
  cpplint issues resolved in vp9_variance_mmx.c
  cpplint issues in vp9_ssim.c

Merge "cpplint issues in vp9_picklpf.c"

Merge "cpplint issues resolved vp9/vp9_cx_iface.c"

Allow sub8x8 intra modes test for alt frame coding

This commit allows sub8x8 intra modes test in the rate-distortion
loop for hd sequences in speed 1 and 2.

For sequence y90n of hd set at 8000 kbps, speed 2 runtime goes
from 207s to 210s. For ped_1080p at 3000 kbps, speed 2 runtim goes
from 336s to 337s. Both are running with 300 frames.

This improves compression performance by 0.24% for stdhd and 0.32%
for hd.

Change-Id: I173ca38a6411565ae6cfadd184c42b2070c5de1f

cpplint issues with vp9_boolhuff.c resolved

Change-Id: I6990c9ab838323d8770dd1f49a25bf3acc4c05c7

Merge "cpplint issues with vp9_temporal_filter.c"

cpplint issues resolved vp9/vp9_cx_iface.c

Change-Id: I4f66d6f1aebe7d47ad01cda9b03c600725240680

cpplint issues in vp9_picklpf.c

Change-Id: I62e631ca95fefbb1a993479a5e3926dc81359fe7

Merge "lint issue  in vp9_psnr.c"

Merge "vp9_encodeframe.c cpplint issues resolved"

Merge "cpp lint issues resolved in vp9_encodeintra.c"

Merge "cpplint vp9_dct.c issues resolved"

Merge "cpplint issues vp9_tokenize.c resolved"

Cleaning up foreach_predicted_block_in_plane() function.

Change-Id: Ibb3d9667eba56621667412f62097aa7a392659c2

cpplint issues with vp9_temporal_filter.c

Change-Id: I695a990689c79d160227975116125b140875aed1

Merge "Adding vp9_get_filter_kernel() function."

Merge "Further clean up of speed 4"

vp9_encodeframe.c cpplint issues resolved

Change-Id: Id9d837e062d9c4a94def4b4ed1f49a67c75d3618

cpp lint issues resolved in vp9_encodeintra.c

Change-Id: Ib6a8360d24f44eeaec12c5055568382a105dc235

cpplint issues in vp9_lookahead.c

Change-Id: I2a98995f0df77d99dc47bda5e41886f014d8843f

cpplint issues resolved in vp9_variance_mmx.c

Change-Id: Idbfabe427fbeab44210f13fec8b6f63f7a4eb0dd

added nolint to function that doesn't seem easy to breakup

Change-Id: I5489b116aea7c510ea5ebbed3c1445f321b05f3e

Giving consistent names to IDCT/IWHT functions.

The idea is to have the following names for each transform size:

vp9_idct4x4_add
  vp9_idct4x4_1_add
  vp9_idct4x4_10_add
  vp9_idct4x4_16_add

vp9_idct8x8_add
  vp9_idct8x8_1_add
  vp9_idct8x8_10_add
  vp9_idct8x8_64_add

etc for 16x16, 32x32

The actual list of renames in this patch:

vp9_idct_add_lossless     -> vp9_iwht4x4_add
vp9_short_iwalsh4x4_add   -> vp9_iwht4x4_16_add
vp9_short_iwalsh4x4_1_add -> vp9_iwht4x4_1_add

vp9_idct_add            -> vp9_idct4x4_add
vp9_short_idct4x4_add   -> vp9_idct4x4_16_add
vp9_short_idct4x4_1_add -> vp9_idct4x4_1_add

Change-Id: I6f43f7437c68dd30cdd05d72e213765578ed30b1

cpplint vp9_variance_sse2.c

Change-Id: Ifce8f5b57a1ea8952e8a67c5b92a127a061899fa

cpplint issues in vp9_ssim.c

Change-Id: I788f1a3004643347ca08d08fc3cb2bb8f0b134d9

cpplint vp9_dct.c issues resolved

Change-Id: Ia21653a447040f1b472d21ebd19103b0558c4b16

cpplint issues vp9_tokenize.c resolved

Change-Id: Id4ec0084641d2ad4def95fb05239455fbc25f9b9

Merge "cpplint issues vp9_encodemv.c"

Merge "cpplint issues vp9_mbgraph"

Fix first pass for non-square blocks

Change-Id: Ic049f0a6ce190f33859118e7b8cfcfe305979102

Merge "Moving all idct/iht functions in one place."

cpplint issues vp9_mbgraph

Change-Id: Iedf9ac460edb31d7c072e2bebd26f2afe8e6089b

cpplint issues vp9_encodemv.c

Change-Id: Icda1d2d7cbfb176884fa6c7d9366a2d60e2994e9

lint issue  in vp9_psnr.c

Change-Id: Ifc7ffc02cfedb47230571298622602609a4e8a70

Merge "Remove redundant second_ref_frame check in sub8x8"

Further clean up of speed 4

Speed 4 still does not give a big gain over speed 3.
This just cleans it up a little from the last patch and comments
out features that do not seem to be giving much benefit.

Change-Id: I5f366e6160e1dbe5dc45cf5eb90cc02712baa1b6

Remove mode_skip_start and mask code for sub 8x8

This code serves no purpose in the re-factored sub 8x8 code.

Change-Id: I5364986224d1a28b71bcb046ec8557a3d14aaa47

Selective masking of split modes.

Allow selective masking of individual split modes rather than
just a single on / off flag.

For speed 2 recovers the large speed loss seen for some derf
clips  in change Ie6bdfa0a370148dd60bd800961077f7e97e67dd4
and a small quality gain.

For speed 1 10 % speed increase observed locally on some derf clips
for minimal quality change.

Change-Id: If86191087b93cbc05351c26c60c7933e2149e485

Missing threshold case for disable split.

In relation to change:
Refactor inter mode rate-distortion search
 Ie6bdfa0a370148dd60bd800961077f7e97e67dd4

sf->thresh_mult_sub8x8[THR_INTRA] = INT_MAX missing;

Change-Id: Ia86b68a5073368a3e2ca124a27b632243b525c8b

Adding assign_mv() function to reduce code duplication.

Change-Id: I2b4e5b842c19f64749b18946ad215c0caa57e7b7

Merge "Replacing duplicated code with get_scan_and_band call."

Adding vp9_get_filter_kernel() function.

Moving INTERPOLATIONFILTERTYPE enum and subpix_fn_table struct to
vp9_filter.h. Adding convenient typedef for subpel kernels.

Function vp9_setup_interp_filters() besides setting xd->subpix.filter_x &
xd->subpix.filter_y has a side effect of also setting scale factors. This
is not required inside decode_modes_b() because scale factors have been
already set by set_ref() calls. That's why replacing
vp9_setup_interp_filters() call with newly created vp9_get_filter_kernel()
call. The behavior of vp9_setup_interp_filters() is unchanged (it
is used from the encoder).

Change-Id: I3f36d3f7cd8d15195a6e2fafd1777cdaf9ecb847

Merge "Reading diff update flag inside vp9_diff_update_prob."