Merge "fix cpplint issue in reconintra"

Merge changes Ia7969baa,Ic5807152,I1c3943cd,I0b5af849,I01cbd1b0

* changes:
  fixed cpplint issue with vp9_scale.h
  vp9_entropymv.c cpplint issues resolved
  cpplint fixes to debug modes
  cpplint issues in vp9_onyx.h
  cpplint issues resolved in vp9_dx_iface.c

Merge "cpplint issue with treedreader"

Merge "Some minor changes/cleanups in rate control"

fixed cpplint issue with vp9_scale.h

Change-Id: Ia7969baac7ffc6d7a0e8e8e83e9252d077a3c5b3

vp9_entropymv.c cpplint issues resolved

Change-Id: Ic5807152cc78127b3f84b5abb4c5f3ef6d06ce65

cpplint fixes to debug modes

Change-Id: I1c3943cd5db6cd8fc759116a3717dba3c030fa0d

fix cpplint issue in reconintra

Change-Id: I934f9cfb96ce4f5f266b025064237875dcd92b3a

cpplint issues in vp9_onyx.h

Change-Id: I0b5af849833ac077bd4de71a24af8f8bd7ec06d6

cpplint issues resolved in vp9_dx_iface.c

Change-Id: I01cbd1b00d8d8e02541b2c29b9e88e690edfcaba

cpplint issue with treedreader

Change-Id: I4036add96dd5e42896c57a80a6ef2b6f27b8224a

Merge changes I29b5bbb9,Iaa6b8ac9,Ibf996de7,Ie1b544e4,I9dea60e3,If71923f4,I6498d2ee

* changes:
  cpplint issue extra line in decodemv.c
  cpplint issue - vp9_idct_blk.c
  cpplint issue in vp9_detokenize.c
  fixed cpplint issue vp9_onyxd_int.h
  cpplint issue in vp9_read_bit_buffer resolved
  cpplint issue vp9_decodeframe.c
  fix cpplint issue in vp9_onyxd.h

Merge "cpplint issues vp9_thread.h"

Merge "cpplint style issue"

Merge "fixed cpplint issues in vp9_onyxd_if.c"

Merge "thumb: Keep whitespace between arguments as before"

Merge "Using is_inter_block and has_second_ref functions."

Merge "Moving from int_mv* to MV* (3)."

Merge "Reusing FRAME_CONTEXT struct to simplify the code."

Merge "Renaming vp9_short_idct10_8x8_add to vp9_short_idct8x8_10_add."

Merge "Removing vp9_get_coef_neighbors_handle function."

Merge "Removing vp9_subpelvar.h from common."

Merge "Fixing warning generated by gcc."

Merge "Removing unnecessary function calls."

cpplint issue extra line in decodemv.c

Change-Id: I29b5bbb9bed7296d0bf7d58ae1e78187ccdc5b34

cpplint issue - vp9_idct_blk.c

Change-Id: Iaa6b8ac967c0000d4632b64ff9709304072d6ef2

cpplint issue in vp9_detokenize.c

Change-Id: Ibf996de79e9c9bbe03b2202d4af11aebc58f9bcc

fixed cpplint issue vp9_onyxd_int.h

Change-Id: Ie1b544e488a5e346a62174bfdeb9b54c34a19083

cpplint issue in vp9_read_bit_buffer resolved

Change-Id: I9dea60e39bc4a51684cfba49c82c3570a2f7b61e

cpplint issue vp9_decodeframe.c

Change-Id: If71923f4821a7bf3372a1ead83baa91fc576977c

fix cpplint issue in vp9_onyxd.h

Change-Id: I6498d2eee0b3f3bbb94787eb0ba72ccfcf8f5f02

cpplint issues vp9_thread.h

apparently we are going to have trouble completely removing lint issue in this file.
It needs a bit more work.   We need to include vpx_config.h to know whether
we need to have multi threading .    and that means vpx_config.h has to come
before the system headers.  ( a violation )

Change-Id: I023feeab1bf5643b79dccc3b80a4a9ad42689e7b
Signed-off-by: Jim Bankoski <jimbankoski@google.com>

thumb: Keep whitespace between arguments as before

Include the whitespace after the first argument's comma in the
optional first argument group.

This fixes a minor style regression in the converted output
since 2a233dd31.

Change-Id: I254f4aaff175e2d728d9b6f3c12ede03846adcf1

cpplint style issue

Change-Id: I550e27b2d40f0e608032e74e1472ceec53c97dc7

fixed cpplint issues in vp9_onyxd_if.c

Change-Id: Ia67e9ed2d5ea79f3dbf1d58f9a187cb18ecd0995

Merge "mips dsp-ase r2 vp9 decoder extend module optimizations"

mips dsp-ase r2 vp9 decoder extend module optimizations

Adopted again to shepherd through system.

Change-Id: If1b742618a95a2198ae2e30eec4b53ad1f7353e5

Removing unnecessary function calls.

Both vp9_init_mbmode_probs() and vp9_zero(cm->ref_frame_sign_bias) are
called inside vp9_setup_past_independence() which called in any case for
encoder/decoder after VP9_COMMON struct creation.

Change-Id: I3724d1a4fb8060101ff0290dd6a158f0b5c57bb4

Reusing FRAME_CONTEXT struct to simplify the code.

Change-Id: Ia455c1900d84a3221e3681e31e15ca86bd03f89d

Merge "Simplify RDMULT and RDDIV derivation"

Renaming vp9_short_idct10_8x8_add to vp9_short_idct8x8_10_add.

Making name consistent with vp9_short_idct8x8 and vp9_short_idct8x8_1.

Change-Id: I99e0be040ec893f9571dcf090e18f98dc58339f5

Properly save neon registers.

Replace current code which corrupts the stack by
duplicate of vp8 code to save and restore neon
registers.

Change-Id: Ibb0220b9aa985d10533befa0a455ebce57a2891a

Removing vp9_get_coef_neighbors_handle function.

Change-Id: I6be72c8b048d1ccc7ef43764cf84c32360098970

Some minor changes/cleanups in rate control

Some small changes to the quantizer mapping functions.
Also includes some cleanups.

Change-Id: I9dea29b24015f6e6697012a0e4d8983049d8e5c7
Results:
derfraw300: +0.106%
stdhdraw250: +0.139%

Merge "Renaming vp9_short_idct10_16x16 to vp9_short_idct16x16_10."

Merge "New way of updating last frame segmentation map."

Merge "d63 intra prediction ssse3 using bytes"

New way of updating last frame segmentation map.

Implementing more natural (and faster) way of updating last frame
segmentation map.

Change-Id: I9fefa8f78e77bd7948133b04173da45edc15a17e

Merge "intel compiler build no longer supports -axSSE2 or -no-prec-div"

Merge "Fix a bunch of TODO from vp9_short_idct32x32_add_neon."

Renaming vp9_short_idct10_16x16 to vp9_short_idct16x16_10.

Making function name consistent with vp9_short_idct16x16 and
vp9_short_idct16x16_1.

Change-Id: I70e54be9e6b9a1dddab0de470686591e96d05517

Simplify RDMULT and RDDIV derivation

Don't divide RDMULT and RDDIV by 100 when RDMULT > 1000. This was
probably done to avoid overflow when the rd cost was stored in a 32 bits
integer but this is not the case anymore. This change will make it easier
to support multiple quantizers per frame.

derf compression gain at speed 0: 0.037%

Change-Id: Ibeeb9b7cfa1a132a7af41bc90fc07a3bba0857f6

Merge "fixed integer overflow warnings"

intel compiler build no longer supports -axSSE2 or -no-prec-div

Change-Id: Id129cf4021903c554ffdc57bf13df9fbb98f800e

Align structs used unit test to 32 bytes

Change-Id: Id36309d405bf7b7e8805a8320f5a95eeb5d240f5

Fixing warning generated by gcc.

vp9/vp9_cx_iface.c:92: warning: type qualifiers ignored on function
return type

Change-Id: I6f130e280e2db261506a4af8ce11fc788ad13198

fixed integer overflow warnings

Jenkins warns on left shift of negative numbers and non-aligned read
of int. This commit fixed the two issues.

Change-Id: I389a7fb6a572c643902e40a4c10fefef94500d2c

Merge "Revert "Align struct to 32 bytes""

Revert "Align struct to 32 bytes"

This reverts commit b5242368f3800e3b658e6ee4567bd41110c80071

Change-Id: I73c7a8bd23bb67f278f86b3fd2ecdc25a39b12b9

Fix a bunch of TODO from vp9_short_idct32x32_add_neon.

- full ASM version, no more C gateway file.
- integrate combine-add with last step of 2nd pass.
- remove a few push/pop pairs.
- some instruction reordering to hide latency.

Change-Id: Ic9d9933c908b65d1bf7ba8fd47b524cda808c9c6

Using is_inter_block and has_second_ref functions.

Change-Id: I60dee58a4fd24d3c4f3c101a49d30e217309f43a

Merge "Correctly set the segment_id prediction flag and context"

Merge "Align struct to 32 bytes"

Merge "Limit mv search range for first pass and mbgraph"

Align struct to 32 bytes

Change-Id: I32fd813af7dd2a7d451f5fda278ea888b181fa8e

Moving from int_mv* to MV* (3).

Change-Id: I9795d0937bc07793c13d067281995e0750f694d9

Merge "Moving from int_mv* to MV* (2)."

Removing vp9_subpelvar.h from common.

Moving all code from that file to vp9_variace_c.c in the encoder.

Change-Id: Ic803d5b4c78d5191e4d25541b3df97337878fc3e

Merge "Replacing txfm with tx."

Merge "Removing unused SUBMVREF_COUNT constant."

Merge "Adding vp9_get_entropy_contexts function."

Merge "Removing redundant 'extern' keyword."

Removing unused SUBMVREF_COUNT constant.

Change-Id: I302ab4603553352a84b57bc89bc9e3d037978d29

d63 intra prediction ssse3 using bytes

byte version of ronalds d63 ssse3 optimizations
(commit: c5a1c8cf3541cf3665fee981b36d22c9fbd4191e)

Change-Id: Ifd3e6d454a2246085f23eabb38518a930321e807

Adding vp9_get_entropy_contexts function.

Change-Id: Ife0dd29fb4ad65c7e12ac5f1db8cea4ed81de488

Replacing txfm with tx.

Renaming txfm_stepdown_count to tx_stepdown_count and max_txfm_size to
max_tx_size.

Change-Id: Ifc173e22c78240e561a57c4c741b64b1b8fc6fef

Merge "Cleaning up vp9_update_nmv_count function."

Cleaning up vp9_update_nmv_count function.

Using best_mv[2] array instead of two separate variables.

Change-Id: Iefa0a41f5c42c42f2c66cef26750da68405f0f25

Removing redundant 'extern' keyword.

Change-Id: Ie51306689c0dc527a8aa12d3984389dd8f360dea

Replacing unsigned char* with uint8_t*.

Change-Id: I99a1880aee015ae16311ba05a31aa307df89bef2

Correctly set the segment_id prediction flag and context

This fix a bug introduced by ac6093d179d8488f90bf10c6d0abcc48be327d28

Change-Id: I0700a4daf7a6a2471074f81a4596352287fb2ac9

Limit mv search range for first pass and mbgraph

Both first pass and mbgraph search use block size 16x16 for motion
estimation. This commit put a limit of motion vector range. The
effective range allows the entire 16x16 with required subpel
interpolation input to be completely outside image border, but
not any further away from image border.

Change-Id: Id70a5ed08be49e70959f064859d72adc7d775d08

Moving from int_mv* to MV* (2).

Updating fractional_mv_step_fp and fractional_mv_step_comp_fp function
types.

Change-Id: I601c4378bc39ac3ffd4e295d9cbd8e1f74829d46

Merge "Remove redundant mode update in sub8x8 decoding"

Merge "Moving from int_mv to MV."

Merge "Replace memcpy with vpx_memcpy"

Replace memcpy with vpx_memcpy

Also removed obselete comment

Change-Id: Iae1664777d76383639c637ee786e0d50fc45819a

Rename defined constants

The change is to better reflect the nature of the constants.

Change-Id: Icabac6e9bceefbdb3f03f8218f88ef75943c30fb

Prevent using uninitialized value in RD decision

INT64_MAX may be assigned as RDCOST when RDCSOST computation is skipped
for speed, this commit to prevent INT64_MAX from being used as real
RDCOST in transform size decision.

Change-Id: I89a945134191bbdea1f1431ade70424ac079eaac

Merge "Change to prevent invalid memory access"

Merge "Adding best_mv[2] array instead of two variables."

Merge "Small tweak in the constant quality parameter"

Merge "vpxenc: Stop writing the WebM FrameRate element."

vpxenc: Stop writing the WebM FrameRate element.

The FrameRate element has been deprecated.

Change-Id: I68ac496e7b33685a100a45d2772e0ff30eb6a811

Merge "Enable per transformed block zero coeffs forcing"

Merge "Calculate rd cost per transformed block"

Small tweak in the constant quality parameter

Improves results a little.

Change-Id: I7bcac02dbb65b43a993445cf557c520197114e5c

Merge "Number of instructions in fdct4_1d_sse2 reduced by two."

Change to prevent invalid memory access

After change of MI context storage , mi_8x8[]  pointer may be null for
a block outside of image border. The commit changes to access the data
only after validation of mi_row and mi_col.

Change-Id: I039c4eb486a228ea9d8e5f35ab9ae6717d718bf3

Number of instructions in fdct4_1d_sse2 reduced by two.

Mathematically the results are the same.

Change-Id: I1c5126cd3ca64e8515ca6331e0989c6f7dd651a0