Fix valgrind read out of bounds error.

MMX variance code in vp8 was reading out of bounds..

TODO(JBB):  The best fix would involve removing duplicate library
functions between vp8 and vp9...

Change-Id: I5722853a6a58d3b55257ff385fa54c773bf98ded

Merge "Cleaning up vp9_cx_iface.c."

Merge "Cleaning up vp9_pick_inter_mode()."

Merge "Reusing swap_block_ptr() function."

Merge "Removing unused fields from twopass_rc struct."

Merge "Hiding vp9_sub_pel_filters_{8, 8s, 8lp} filters in *.c file."

Merge "Making vp9_initialize_dec() static."

Merge "Rewrite encoder input format handling."

Merge "Add a TODO"

Merge "Moving PC_TREE from MACROBLOCK to VP9_COMP."

Merge "vp8: Add increase_denoising parameter to denoiser."

Merge "Removing MACROBLOCKD dependency from loop filter."

Merge "Correct HAVE_NEON_ASM define"

vp8: Add increase_denoising parameter to denoiser.

Change-Id: I96ed73e109c4f89dd06f3583cf7ecf9277401fae

Add a TODO

Change-Id: I16bf93d40e9b345705b49bf09dd4b6996b513a83

Rewrite encoder input format handling.

The previous approach can't scale to all of or new formats. This also
fixes YV12 input.

Change-Id: Ic4ad333770bbbd0e921f519199b57862afcbe1ab

Moving PC_TREE from MACROBLOCK to VP9_COMP.

Because PC_TREE is encoder-level data, not MACROBLOCK-level data.

Change-Id: I4f620c0781acd3a2744860610117e74948e0b2b5

Cleaning up vp9_cx_iface.c.

Marking unused parameters with (void), adding consts, fixing formatting.

Change-Id: I8ac1e6606c0f2673f78bc41830e672a680ffed02

Removing MACROBLOCKD dependency from loop filter.

Change-Id: I9ef40f3d95ab8f94f69e92ea25678a40956bc1ce

Merge "Removing redundant decoder_init flag."

Merge "Fix post-processor macros & remove vizualization"

Correct HAVE_NEON_ASM define

These optimizations are currently disabled.

Change-Id: I19c58c9cb82d017638b86196641b9e001dfa798b

Merge "Remove intermediate step in vp8_dequantize_b"

Merge "cleanup -wextra warnings:"

Merge "Reuse precalculated result"

Reuse precalculated result

Change-Id: Iff9efff6c9cb41f833cee40eae014bd4489a87d0

set_map.c: cleanup -wextra warnings

The commit changed to use memset for initialiazation of non-trivial
strucutures, where initialization using {0} caused warnings. Also,
removed {0} initializations where appropriate initialization calls
are in place.

Change-Id: Ifd03e34aa80688e382124eb889c0fc1ec43c48e6

Merge "Removing redundant "8x8" suffix from MODE_INFO vars."

Merge "vp9_rdopt.c: cleanup -wextra warnings"

Merge "vp9_tokenize.c: cleanup -wextra warnings"

cleanup -wextra warnings:

  vp9_decoder.c
  vp9_dthread.c

Change-Id: Iaafe941545db98e9e3559096a955894646084ac2

Merge "vp9_firstpass.c: clean -wextra warnings"

Merge "Removing unused img_setup field."

Removing redundant decoder_init flag.

Change-Id: Ieee7a7e3c40d6bcc9fa4df8d10ee9620995aa691

Merge "vp9_pickmode.c: cleanup -wextra warnings"

Merge "Build armv7a-only code"

Merge "Revert "Remove Wextra warnings from vp9_sad.c""

Merge "AVX2 To VP9 Block Error Optimization"

vp9_firstpass.c: clean -wextra warnings

Change-Id: Ic488fe6edbc119f475763d72a85809499df60106

vp9_pickmode.c: cleanup -wextra warnings

Change-Id: I07e39f04fb11d2673722651fc78d0e9d22b1f557

vp9_rdopt.c: cleanup -wextra warnings

Change-Id: I1f87c689cad2224181d40a7d00f5c1064ceefa4b

Reusing swap_block_ptr() function.

Change-Id: I51ae33fcaaa7330b68493a782ec6ef02539fda71

Removing redundant "8x8" suffix from MODE_INFO vars.

Change-Id: I7ed7fecc959c6598ff98895f1a5cf7e11ac1615f

Removing unused fields from twopass_rc struct.

Change-Id: Iaece070e9d1305ac3d8df9d3431fefa6e20ac0ec

Cleaning up vp9_pick_inter_mode().

Change-Id: I21bff31e58e9078b4ffcbd2027cc52366843ff23

Merge "Refactor calling loopfilter code."

Refactor calling loopfilter code.

This change is mainly for a follow CL that will refactor the
decode_tiles.

Change-Id: I52de6f8dbada75a64d9a94ebb5975136ed0960b4

Fix post-processor macros & remove vizualization

Make all post-processor code conditionally
compilable based on the CONFIG_VP9_POSTPROC
macro.

Also, remove the vizualization code from VP9
since it is out of date and will not compile.

Change-Id: I1e9e13a09ecd43e9a3f3704c175ae8cd258ababd

Revert "Remove Wextra warnings from vp9_sad.c"

This reverts commit 7ab9a9587b96db4edce6be916c1f02297a9555ff

Nightly test http://build.webmproject.org/jenkins/view/libvpx-nightly-tests/job/libvpx%20unit%20tests%20(valgrind-2)/arch=x86_64-linux-gcc,filter=-*VP8*:*Large.*/276/console

Failed

This patch did not address all the assembly issues
some of the vp8 assembly counts on 5 arguments being passed in to this function:

one example : vp8_sad8x16_wmt

Please address or split this into vp9 and vp8 patches.

Change-Id: I78afcc171649894f887bb8ee3c66de24aaddc7ca

Merge "vp9_decodeframe.c: cleanup -wextra warnings"

Merge "vp9_ratectrl.c: cleanup -wextra warning"

Merge "Silence unused parameter warnings."

Merge "Clear unused parameter warning."

Merge "vp9_quantizer.c: cleanup -wextra warnings"

Merge "vp9_mbgraph.c: cleanup -wextra warnings"

Removing unused img_setup field.

Change-Id: I152e54fa1acceea5ec809d5bf7bfa7f4ef2b9acf

Making vp9_initialize_dec() static.

Change-Id: I831fe91dfadf4e89f5bbba6ab7a9917d8dd2ed55

Hiding vp9_sub_pel_filters_{8, 8s, 8lp} filters in *.c file.

Change-Id: Id401da740b0a0141caaef9e1bcccd981e5cef4a4

Merge "Rename vpx_temporal_scalable_patterns.c"

Merge "Moving frame_coef_probs from VP9_COMP to local scope."

Merge "Cleaning up vp9_context_tree.c."

Remove intermediate step in vp8_dequantize_b

With the intrinsics it is no longer necessary to have a stub/helper
function.

Change-Id: I3695961c3c94f1bb750d3b7b29716e509ebba482

Build armv7a-only code

Allow disabling the more generic NEON code.
Use filtered option to disable rtcd code.

Change-Id: Icb4500c1a2bac16eed3c5e3ec0c35e92e6bbbb9f

AVX2 To VP9 Block Error Optimization

vp9_block_error_sse2 can only handle 16 bytes at a time but
the function requires to handle a sequence of 32 bytes at a time
so each 16 bytes is handled in a different register.
With AVX2 optimization the 32 bytes can be handled in one register instead
of two in the SSE2
The vp9_block_error was optimized by 85%.
The user level was optimized by 1.2%

Change-Id: Ia8fffe60e61eff7432a5fbd538757894f6c319fd

Merge "Revert "VP8 for ARMv8 by using NEON intrinsics 06" This reverts commit 81ad047ee57ecb0e2c1ee4dcebda54a44ea54ae9. Revert "VP8 for ARMv8 by using NEON intrinsics 15" This reverts commit 727af7cebe3698b8493ba6c1360b0a6606c310fb."

Rename vpx_temporal_scalable_patterns.c

New name: vpx_temporal_svc_encoder.c
Also, update comment to note that example supports VP8 and VP9.

Change-Id: I6fffab81296f918ebca740192a5c609593852dff

Merge "Revert "Revert "Remove struct params from vp8_denoiser_filter"""

Revert "Revert "Remove struct params from vp8_denoiser_filter""

This reverts commit 06e6d56fa138d84759e8bdfd4c721ead000051b4

Change-Id: If95598385b693945d6b144d03b6da8f6a57dac98

Merge "Tune minq index table for P frames in one_pass_cbr coding mode"

vp9_ratectrl.c: cleanup -wextra warning

Change-Id: I7ca46fa26acd80a21210cb8d9584ad812cb995bf

vp9_tokenize.c: cleanup -wextra warnings

Change-Id: I85854f1ab2d60feea0bfe6b2141f89a998777ecb

Merge "Fix int compared to unsigned int warnings."

Merge "Simplify 2 pass KF bitrate allocation"

Merge "Further two pass clean up."

Merge "Remove Wextra warnings from vp9_sad.c"

vp9_decodeframe.c: cleanup -wextra warnings

Change-Id: I0315cea6a5e58182bc2556e9825ec2ef0b1480c3

vp9_quantizer.c: cleanup -wextra warnings

Change-Id: If5a3c48a8c554018a5d63c1541a2900f15767a00

Merge "Silience -wextra warnings in vp9_reconintra.c"

Clear unused parameter warning.

Caused by a parameter which is only used under
#if CONFIG_VP9_POSTPROC.

Change-Id: I2c0979d215d45f56da1a38bb13aa214bde52a640

vp9_mbgraph.c: cleanup -wextra warnings

Change-Id: Ia6e2c2741adbf45f98a447dbb401506f95a2b0c8

Merge "Rename vp9_spatial_scalable_encoder."

Merge "Add test for the twopass_encoder example."

Merge "Add test for the resize_util example."

Silence unused parameter warnings.

The various motion search functions share a
common function prototype. In the case of
vp9_full_range_search() two of the parameters
are not needed.

Change-Id: I0e190af54a3b3f276409f20e8ec55912f9b0b798

Fix int compared to unsigned int warnings.

Also delete vp9_set_roimap() which is not used in VP9.

Change-Id: I48c60b1dc8794c6a234f9c3d95e2debb7bdb3fd5

Merge "Clean up two unused parameters."

Silence unused parameter warnings.

Remove two unused parameters in the function
vp9_refining_search_8p_c().

Change-Id: Ic192734586291cf5400926eeb8e720e69d40835c

Clean up two unused parameters.

Unused parameters to encode_without_recode_loop()

Change-Id: Ia94ce09199cd2419f26964eb5b53db2b7718cf03

Simplify 2 pass KF bitrate allocation

Simplify the calculation of KF bitrate in similar way
to previous patch for GF/arf.

This has no impact on derf or std hd sets but gives a
small net gain of ~0.1% for yt and yt-hd sets.

Change-Id: Ida64ac1428d9c2a62adb67056fadbf0180eff030

Further two pass clean up.

The variation in boost calculation for gf and arf groups
is not significant enough to justify the extra complexity.
Also removed some other spurious code that no longer
has much material impact.

The handling of the rare case, where the boost bits
number is less than the number of bits a that would
be allocated if a frame was not boosted, will be dealt
with in a subsequent patch.

This change actually helps on all sets a little by
~0.1% - 0.2% with slightly bigger gains on SSIM.

Change-Id: Id42c1ac22a80a8c4993cfa0e51bc733eb9ed4f75

Remove Wextra warnings from vp9_sad.c

As a side-effect, the max_sad check is removed from the
C-implementation of VP8, for consistency with VP9, and to
ensure that the SAD tests common to VP8/VP9 pass.
That will make the VP8 C implementation of sad a little slower
but given that is rarely used in practice, the impact will be
minimal.

Change-Id: I7f43089fdea047fbf1862e40c21e4715c30f07ca

Add test for the resize_util example.

Change-Id: I97c052a35a1d54455b91c269fd6a9a094fa3db7f

Merge "Moving loopfilter call to vp9_decode_frame()."

Merge "vp9_convolve.c: cleanup -wextra warnings"

Rename vp9_spatial_scalable_encoder.

Rename it to vp9_spatial_svc_encoder.

Change-Id: I756936039df5bf6bea2d7525a7edeac01df94352

Merge "Adding get_tile_buffers() function."

Merge "Fixing constant value used to calculate frame pts and duration."

Merge "Inlining setup_key_frame() and setup_inter_frame()."

Revert "VP8 for ARMv8 by using NEON intrinsics 06"
This reverts commit 81ad047ee57ecb0e2c1ee4dcebda54a44ea54ae9.
Revert "VP8 for ARMv8 by using NEON intrinsics 15"
This reverts commit 727af7cebe3698b8493ba6c1360b0a6606c310fb.

This exposes a bug in gcc 4.9 regarding register allocation. Will reland
when 4.9 is fixed.

Change-Id: I2d8a04e4edde93719280e41550f4c0765608ec4d

Silience -wextra warnings in vp9_reconintra.c

The warning messages complained that there are unused arguments
in a few prediction modes. This structure was designed on purpose,
such that a wrapper function can cover all prediction mode cases
and make them readily accessible as an pointer array.

This commit silences such warnings.

Change-Id: I7036b6bdb70747e5327d8f6fceb154f100abc4c0