Merge "Cleaning up vp9_init_dsmotion_compensation()."

Refactoring motion search libs

The core motion estimation fucntions all return sad now consistently.
The only exception is vp9_full_pixel_diamond(), however the core diamond
and refining search routines called from vp9_full_pixel_diamond() also
return SAD. If variance of pred error + mv cost is desired it must be
calculated explicitly outside these functions. For very fast encoding,
hopefully this will eliminate some redundant computations.

Also suggests reimplementing FAST_HEX with the vp9_pattern_search
framework. It is not exactly the same as the existing FAST_HEX, but
performance is slightly better and speed is very similar. Enables
removing a lot of duplicate code.

Change-Id: I152736393438c25bdf7e96b37cbb8ce330f4f94a

Merge "Adding get_tx_type() instead of get_tx_type_{8x8, 16x16}."

Merge "Moving FRAME_CONTEXT & FRAME_COUNTS to vp9_entropymode.h."

Merge "Adding consts and cleaning up vp9_rdopt."

Merge "cost_coefs approximation speed up"

Merge changes I67b2b0d7,I24c024d1,I3703d739,Ia6f669a9

* changes:
  nestegg: packet_track: fix uint64->uint32 warning
  nestegg: fix track_scale double->uint64 warning
  nestegg: fix track_number uint64->uint32 warnings
  nestegg: add ne_get_uint32 convenience function

Merge "Enforce intra_y_mode_mask for inter rd."

Merge "Refactor ZEROMV equivalence"

Merge "Reset search_method for non-rd pick mode"

Adding get_tx_type() instead of get_tx_type_{8x8, 16x16}.

Change-Id: I4a54b12e5229705222c5a101258b9d1f81e2948d

Adding consts and cleaning up vp9_rdopt.

Change-Id: I9423b543e1be414e5c9e10480b813f06e6b88f8a

Merge "Merging error-resilient example into simple_encoder."

cost_coefs approximation speed up

This patch adds a new speed feature which doesn't do the rather
expensive entropy context lookup or save to the table,  while
doing costing.

The speed up on desktop36p.y4m is around 10% other clips much less.

On the RTC test set this was + 1% in overall datarate.

Change-Id: Ia5144bbf45270671e7be9c8e4055369909e2f738

Refactor ZEROMV equivalence

This gets more accurate mode hit stats. It's also the first step to
handling ZEROMV not being allowed more intelligently.

Change-Id: I5de6734507b5177bf73e9ddbad923f218c39f3e4

Reset search_method for non-rd pick mode

Reset FAST_HEX search_method, which was lost because of code merging.

Change-Id: Ic2b2cb9ec2f1751e96f7db0f9226310f605fcc83

Enforce intra_y_mode_mask for inter rd.

intra_y_mode_mask is already enforced for the sub8x8 case.
intra_uv_mode_mask is already enforced for all sizes.

Change-Id: Ia9dd14701cb49873c2e8f24eb5f8b255eaf76a1f

Merge "Cleanup conditions in vp9_rd_pick_inter_mode_sb()."

Merge "Clear implied conversion to int64_t warning."

Merge "Removing VP9D_PTR."

Merge "AVX2 SubPixel AVG Variance Optimization"

Merge "vp9_lookahead_push(): remove unused parameters"

Merge "vp9_decodeframe.c: remove unused params"

Merge "Silence int64 to int warning"

Merge "vp9_mbgraph.c: remove unused parameters"

Merge "temporal_filter_find_matching_mb_c(): remove a param"

Merge "vp9_prob_diff_update_savings_search_model(): remove params"

Clear implied conversion to int64_t warning.

Clear warning caused by implied conversion from int32 to int64.

Change-Id: I473b37a54984cbfb22702eb3f712465881ff86e8

Silence int64 to int warning

Change-Id: Ic97ddd5143425a7b63fac4e40d9740a10a0491ad

Merge "vp9_firstpass.c: remove unused functions & params"

Merge "Sorting includes in vp9_firstpass.c."

vp9_mbgraph.c: remove unused parameters

Change-Id: I5a01d0dccc9afc87e2adbb5ddc7d44379fe1f125

temporal_filter_find_matching_mb_c(): remove a param

There is one parameter that is never used, therefore is removed.

Change-Id: I595722e7c5731534c72831315359e4dce9c21763

vp9_prob_diff_update_savings_search_model(): remove params

Two paramemters are not in use, therefore are removed.

Change-Id: I41eb601b7b4bf2feedb13b3f240315656d8694f9

vp9_lookahead_push(): remove unused parameters

Change-Id: Ia319bd924401677b5c30b0f46d1ab2be6a1f446e

vp9_decodeframe.c: remove unused params

Change-Id: Ie83e18802234199491dbb6e90886739814558f96

vp9_firstpass.c: remove unused functions & params

Change-Id: I111c5bd0dd47db7b1f6952c18056bee548bb42e9

vp9_encodeframe.c: remove unused function params

Change-Id: I6d47618e8aee6e6c2fe1cf0b8bb4da0b2a0bab66

Remove dec_build_inter_predictors() parameters

There were two parameters not in use, this commit removed them.

Change-Id: Ia03a73b9a2521400bed539df45574e34214ed93a

Remove unused parameters from update_state_rt()

In addition, a local variable was renamed to avoid confusion.

Change-Id: Id4c497f9cfa219e8a414aa9fee9a85af5f147249

Remove vp9_create_common()

The function has evolved over time, now only calls vp9_rtcd(), so this
commit removes the function and changes to call vp9_rtcd() directly.

Change-Id: I8cfa6190daa4b28f6f3d1e11bb3a07f9c95322bf

Remove a parameter from write_selected_tx_size()

The parameter was never used.

Change-Id: I99fd9aad697917ddbe3d10c71cae344228fafc31

AVX2 SubPixel AVG Variance Optimization

Optimizing 2 functions to process 32 elements in parallel instead of 16:
1. vp9_sub_pixel_avg_variance64x64
2. vp9_sub_pixel_avg_variance32x32
both of those function were calling vp9_sub_pixel_avg_variance16xh_ssse3
instead of calling that function, it calls vp9_sub_pixel_avg_variance32xh_avx2
that is written in avx2 and process 32 elements in parallel.
This Optimization gave 80% function level gain and 2% user level gain

Change-Id: Iea694654e1b7612dc6ed11e2626208c2179502c8

nestegg: packet_track: fix uint64->uint32 warning

Change-Id: I67b2b0d722424df9938d7c26f9d3478592bc813a

nestegg: fix track_scale double->uint64 warning

Change-Id: I24c024d1b328b9c34289c125c501d2fdc246e92e

nestegg: fix track_number uint64->uint32 warnings

ne_read_block/ne_find_cue_position_for_track/nestegg_get_cue_point
in the use of ne_map_track_number_to_index
+ add a check to ensure it doesn't exceed the type bounds

fixes:
./third_party/nestegg/src/nestegg.c|1322| warning C4244: 'function' :
conversion from 'uint64_t' to 'unsigned int', possible loss of data

Change-Id: I3703d739dcf9a2d4d8e2b704e957e5e3fd80dca0

nestegg: add ne_get_uint32 convenience function

- avoids downcasting from uint64_t in multiple locations

Change-Id: Ia6f669a9ee81336ed1ef29a6a79240d5905acab5

Merge "normalize perl shebangs"

Merge "Fix unused parameters in vp9_extend_frame_borders"

Merge "fix vp9 only build"

Merge "Replacing int_mv with MV."

Merge "Removing unnecessary casts from quantization code."

Merge "Using ref_mv[2] instead of two separate variables."

Fix unused parameters in vp9_extend_frame_borders

Change-Id: I7255b3bc47d760333f58ac4878becbcc8ad30967

Merge "Sorting includes in vp9_rdopt.c."

fix vp9 only build

Change-Id: I7a1a7190dc32c65846a897a3162a02d63e59719f

Merge "Adds speed 8 to vp9 as reference"

Using ref_mv[2] instead of two separate variables.

Change-Id: I909a81991aa0cc18afe6526542c7513837445213

Sorting includes in vp9_rdopt.c.

Change-Id: I7e2591b4b54e0eac8521e128c7d97f62d2057ed8

Merge "Sorting and eliminating includes in vp9_pickmode.c."

Moving FRAME_CONTEXT & FRAME_COUNTS to vp9_entropymode.h.

Change-Id: I1fe71e35b1e44da693b43d26607abb33efd56820

Merge "Fixing include order in vp9_quantize.c"

Merge "Fix potential invalid partition size use"

Adds speed 8 to vp9 as reference

Adds a speed 8 to VP9 where only the nearestmv (0 mv) is searched.
This seems to be about the same speed as vp8 speed 5.

Adds a new speed feature to disable inter modes based on a mask for
each blocksize.

Adds code for having lower complexity motion search methods
in nonrd pick mode function, even though speed 7 still uses DIAMOND
search for now.

Also uses HEX search for speed 6 rather than FAST_HEX which improves
psnr by 0.56% without any noticeable speed drop (tested on gipsmotion).

Change-Id: Ic13176572dbd3aed5884a26786940a4b1bbd8a75

Merge "Skip some mode SAD calculation in non-RD mode"

Replacing int_mv with MV.

Change-Id: Ib68d4a2de838254a66272324b909292048cbe60d

Removing unnecessary casts from quantization code.

Change-Id: I64172710654e95a90ee754d14d7104337d28010f

normalize perl shebangs

don't require perl to be in a fixed location

Change-Id: Icc8b6113a2a3626f847fe46409334a03f1db5c85

Merge "[svc] Fix "possible loss of data", "division by zero" and "index 5 out of bounds for type" compiler warings"

Merge "Cleaning up vpx_codec_get_cx_data() function."

Fixing include order in vp9_quantize.c

Change-Id: Ic32eb103d0d7f98c0a16c4e7bdec117faf05df02

Merge "Cleaning up vp9_quantize.c."

Sorting and eliminating includes in vp9_pickmode.c.

Change-Id: Icc057b819d80c608a6003fd7ef58aefed7a15824

Sorting includes in vp9_firstpass.c.

Change-Id: Iee4f8443971fe2b21762d87f0be5751329768b5a

Merge "Adding get_y_mode() function."

Merge "Cleaning up vp9_bitstream.{c, h} files."

Merge "Adding consts to arguments of vp9_block_error()."

Merge "Cleaning up encode_ref_frame() to match read_ref_frames()."

Fix potential invalid partition size use

For blocks at frame boundary, the selected block size sometimes needs
to be smaller than that was first given. This commit forces such block
size change only between square blocks, so as to avoid the potential
use case containing 32x16 + 16x8 + 16x8, for 1080p sequences.

Local test suggested no visible coding speed difference. Borg test
reveals no difference in terms of compression performance.

Change-Id: Ie8de87f3c6febc3acf11b4cbfdf2077f9f6def52

Merge "Moving get_tx_eob() from common to encoder."

Merge "Removing unnecessary clamp() call."

Merge "Explicit lossless handling in rd_pick_intra4x4block()."

Cleaning up vp9_init_dsmotion_compensation().

Change-Id: Id5b56c7bc5671859f54ae69693064804d3fadd98

Merge "Replacing int_mv with MV and adding const qualifiers."

[svc] Fix "possible loss of data", "division by zero" and
"index 5 out of bounds for type" compiler warings

Change-Id: I414d39198cb02d1b8e2330002ed5fe062aaea95e

Merge "rtcd.sh: sort functions by name"

Cleaning up vp9_quantize.c.

Change-Id: I9a38af32f16f196b83dd69755eafb9543edf5691

Merge "Fix a bug in neon that has not save and restore q4-q7 registers."

Skip some mode SAD calculation in non-RD mode

This commit checks if the motion vector associated with the current
mode has been computed in previous mode tests. If possible, skip the
redundant reference block generation and SAD calculation in the
non-RD mode decision process.

For test sequence pedestrian_area 1080p, the runtime goes from
24261 ms to 23770 ms. This does not change compression performance.
The speed-up is mostly around places with consistent motion.

Change-Id: I97be63c6a2d07c57be26b3c600fbda3803adddda

Merge "Enable using fast HEX search in non-rd pick mode"

Merge "vpx_scale_rtcd.sh: fix conditional"

Merge changes I6e7d6fa5,Ie033bbf2,I08fa0e90,I8612f197

* changes:
  cosmetics: test.mk: sort source lists
  test.mk: fix test availability
  sad_test: (arm) fix vp9-only build
  cosmetics: sad_test: add a few comments

rtcd.sh: sort functions by name

this will aid in output comparison with other generation systems

Change-Id: Ifb66321e45ab9fdd9eaeb5e0ea8b3d0632efe7ce

vpx_scale_rtcd.sh: fix conditional

previously the scale functions would always be include regardless of the
CONFIG_SPATIAL_RESAMPLING setting.

Change-Id: Ifbccf47b20689b5dd61bb3ddccd5c013297b4e05

Merge "configure: remove x86inc debug message"

Cleanup conditions in vp9_rd_pick_inter_mode_sb().

Change-Id: I4e5ca10f7aef2d37fda835b369828d0bbff7c8a0

Merge "Precompute vp9_rd_pick_inter_mode_sb loop escape conditions."

Merge "Reusing existing mem_{get, put}_be32() functions."

Removing VP9D_PTR.

Change-Id: I17276e25db4592ffeff0961dd9eeaabe4bde110c

Removing unnecessary clamp() call.

Change-Id: Iaaa16b4b2c581eaeb9e4ecfcfe60f98b8a0fa40b