Fix a few indent format issues in buffer defs

Change-Id: Iac55891ac9e6f13718c9f822aa099b5ca491832a

Remove unused comment

Change-Id: I2d96940fae4c7a16661a43c2bf6907d8b1c1a127

Re-design all-zero-coeff block index buffer use

Use the zcoeff_blk buffer of PICK_MODE_CONTEXT to store the indexes
of all-zero-coeff block of the current best mode. Remove the temporary
buffer best_zcoeff_blk defined in the rate-distortion optimization
loop. This improves the speed performance by about 0.5% in all speed
settings.

Change-Id: Ie3e15988ddfa581eafa2e19a8228d3fe4a46095c

Move token_cache from cost_coeffs to MACROBLOCK

This commit moves token_cache buffer into macroblock struct, instead
of defining as a local variable in cost_coeffs. This avoids repeatedly
re-allocating memory space in the rate-distortion optimization loop.

The runtime at speed 0 reduces:
bus 2000kbps, 161692ms to 159951ms
football 600kbps, 229505ms to 225821ms

Change-Id: If7da6b0b6d8c5138a16271a33c4548fba33d8840

Merge "Masking intra mode choice adaptively"

Merge "Removing vp9_tree_p typedef."

Merge "Removing vp9_idct4_1d_sse2 function."

Masking intra mode choice adaptively

The commit changes to mask available intra prediction modes for test
based on prediction block size.

With this patch, encoding time of CpuUsed 2 reduces from 10% to 20% for
HD clips with a compression drop of 0.2%

Change-Id: I65f320f1237c0f5ae3a355bf7caf447f55625455

Merge "Code cleanup"

Merge "Restore mode skip feature in sub8x8 rd loop"

Code cleanup

Minor code cleanup.

Change-Id: I47c1f794842d4570bb39cfd23b80f54f5606bba6

Merge "Experimental rate control change."

Merge "Disable recode loop."

Merge "SSE2 8-tap sub-pixel filter optimization"

Experimental rate control change.

When the codec in VBR (or cq) mode hits its max q limits and is
struggling to hit a target bandwidth, the bit target per frame collapses.

In the first instance normal frames cap out at the maximum allowed
Q and then the ARF and GFs do the same. This latter behavior is not
generally desirable as GFs and ARFs are only effective from a quality
and data rate perspective if they have at lease some level of -Q delta
compared to the surrounding frames.

In this patch I define a separate max Q for GFs and ARFs that is
derived from but somewhat lower than that defined for normal frames.
In effect there is a minimum Q delta that will always be available for
GFs and ARFs regardless of the target rate and MAXQ setting.

This may of course mean that the absolute lowest rate obtainable for
a given clip is somewhat higher.

Change-Id: I268868b28401900d0cd87e51e609cd3b784ab54a

Disable recode loop.

For VBR coding disable the recode loop for speeds > 0.

Results pending.

Change-Id: I2cd9a87c3fcbe39c05b954798d0671a4ca62c37f

Adjustment to allowed range in resize unit test

Change-Id: I5222e3db2627a3a9f7fc34f2ab4554aa5807ed51

Removing vp9_tree_p typedef.

It is used only two times and it is more clear to use real type instead
of typedef.

Change-Id: Idc25c16504c3da4d040e0cdb33a2987631bb6a5b

Removing vp9_idct4_1d_sse2 function.

We have two SSE2-optimized functions for idct4_1d:
  vp9_idct4_1d_sse2 <-- removing this one
  idct4_1d_sse2

vp9_idct4_1d_sse2 was used only by the following functions which already
have SSE2 optimized variants:
  vp9_idct4x4_16_add_c   -> vp9_idct4x4_16_add_see2
  idct8_1d               -> vp9_idct8x8_{16, 10, 1}_see2
  vp9_short_iht4x4_add_c -> vp9_short_iht4x4_add_see2

Change-Id: Ib0a7f6d1373dbaf7a4a41208cd9d0671fdf15edb

d207 intra prediction ssse3 using bytes

byte version of ronalds d207 ssse3 optimizations
(commit: f891f84d3ba9345b0074e682f0fea09b8ddf4f1e)

Change-Id: If15f71a589ea16f78ac86a501b0c5c6231dc9af1

Merge "Giving consistent names to IDCT 32x32 functions."

Merge "Consistent names for FDCT functions."

Merge "d153 intra prediction (32x32) ssse3 using bytes"

SSE2 8-tap sub-pixel filter optimization

To ensure fast encoding/decoding on devices without ssse3 support,
SSE2 optimization of sub-pixel filters was done. Test using 1080p
clip showed the decoder speeds were ~70fps with ssse3 filters, ~60fps
with sse2 filters, and ~15fps with c filters.

Change-Id: Ie2088f87d83a889fba80a613e4d0e287aadd785c

Merge "Fix typo in comment message"

Merge "change to avoid out-of-range computation"

Merge "Re-design rate-distortion cost tracking buffers"

Fix typo in comment message

Change-Id: Ifef756a3a91423bb9f5411f06fa092027be21ecf

Consistent names for FDCT functions.

Renames:
  fdct4_1d   -> fdct4
  fadst4_1d  -> fadst4
  fdct8_1d   -> fdct8
  fadst8_1d  -> fadst8
  fdct16_1d  -> fdct16
  fadst16_1d -> fadst16

"_1d" suffix is redundant, so removing it. The same will happen with idct
in the next change sets.

Change-Id: Ibf421cd2f569146c6079269df7a31819c098265e

Giving consistent names to IDCT 32x32 functions.

Renames:
  vp9_short_idct32x32_add   -> vp9_idct32x32_1024_add
  vp9_short_idct32x32_1_add -> vp9_idct32x32_1_add
  vp9_idct_add_32x32        -> vp9_idct32x32_add

Change-Id: Id85306f5814bac6c47463a6b5901a93082510666

Re-design rate-distortion cost tracking buffers

This commit re-designs the per transformed block rate-distortion
costs tracking buffers. It removes redundant buffer usage, makes
the needed context memory allocation per VP9_COMP instance and
reuses the same buffer sets inside the rate-distortion optimization
search loop, thereby avoiding repeatedly requiring memory space.

It reduces speed 0 runtime:

bus at 2000 kbps from 166763ms to 158967ms,
football at 600 kbps from 246614ms to 234257ms.

Both about 5% speed-up. Local tests suggest about 2% to 5% speed-up
for speed 1 and 2 settings. This does not change compression
performance.

Change-Id: I363514c5276b5cf9a38c7251088ffc6ab7f9a4c3

change to avoid out-of-range computation

Change-Id: Id5e31833a0ef40de9f64c2f5674af7083233bf14

Merge "Removing inv_txm4x4_1_add and inv_txm4x4_add function pointers."

Merge "Moving all scan/iscan code into separate vp9_scan.{h, c} files."

Merge "Giving consistent names to IDCT 16x16 functions."

Merge "Adding const to several pointers."

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%

Adding const to several pointers.

Change-Id: I7231589bda71d0d23c730283febd5bb58585a0da

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

Removing inv_txm4x4_1_add and inv_txm4x4_add function pointers.

We already have itxm_add member in MACROBLOCKD structure. Both
inv_txm4x4_1_add and inv_txm4x4_add are just its special cases for
different eob values. But eob logic is already implemented in
vp9_iwht4x4_add and vp9_idct4x4_add (that's why also removing
inverse_transform_b_4x4_add).

Change-Id: I80bec9b6f7d40c5e5033c613faca5c819c3e6326

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"

Giving consistent names to IDCT 16x16 functions.

Renames:
  vp9_short_idct16x16_add    -> vp9_idct16x16_256_add
  vp9_short_idct16x16_10_add -> vp9_idct16x16_10_add
  vp9_short_idct16x16_1_add  -> vp9_idct16x16_1_add
  vp9_idct_add_16x16         -> vp9_idct16x16_add

Change-Id: Ief8a3904de78deab0f4ede944c4d0339c228cfc3

Restore mode skip feature in sub8x8 rd loop

This commit restores the mode skip feature in the sub8x8 rd loop.

Change-Id: I5496ee32053f572b8961b549e9ecd4f1360824de

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

Moving all scan/iscan code into separate vp9_scan.{h, c} files.

Now we have entropy code separate from scan/iscan code. The next step
in future is to move iscan code from common part to the encoder.

Change-Id: Id9732f7d80aec00af35c1d58d1137c4c96c91451

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"

d153 intra prediction (32x32) ssse3 using bytes

Change-Id: Ie2c0d84ff9f6294084d65f4380e1f30c09e681c9

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"