Merge "AVX2 Convolve Optimization"

Merge changes I121fe66c,I94f57f8a

* changes:
  Remove some cases for turning off arf.
  Modified Aq1 and Aq2

Merge "vp9_mbgraph.c static analysis issues resolved."

Merge "vp9_temporal_filter.c static analysis issue resolution"

Merge "vp9_pickmode.c static analysis issues resolved"

Remove some cases for turning off arf.

Flipping arf on and off too often is hurting some clips.
This change makes no difference for 50-75% of our test
clips but helps some by a big margin. (eg. std-hd crew
by 6% and one of the YT and YT-hd clips by 14%)

Average improvements for 2 pass, speed 2 (psnr,ssim)
are as follows:-

derf  0.165%, 0.210%
yt  1.210%, 1.464%
yt-hd 1.189%, 1.471%
std-hd 1.031%, 0.886%

Change-Id: I121fe66cfb4a62d384b23b484a7d648789641969

vp9_mbgraph.c static analysis issues resolved.

Change-Id: I6ec6e072fcd59f9b5fac4ce221a86f9078103ae3

vp9_temporal_filter.c static analysis issue resolution

Change-Id: I08a4364672cf8171932e8e85fb74fcbfa4b27d2d

Merge "vp9_rdopt.c static analysis issues resolved"

vp9_pickmode.c static analysis issues resolved

Change-Id: I716ea2967c6771d7adea571406f315a3197eab93

Merge "static analysis warning in vp9_ratectrl.c"

Merge "vp9_mcomp.c static analysis issues"

Merge "clear out static analysis warning"

Merge "clean out static analyzer warnings vp9_encodeframe.c"

Merge "Adding and reusing vpx_img_plane_{width, height}() functions."

AVX2 Convolve Optimization

Two convolve functions were optimized for AVX2:
1. vp9_filter_block1d16_h8
2. vp9_filter_block1d16_v8
vp9_filter_block1d16_v8 was optimized for AVX2 by reducing the number of
loop strides by half, two strides were processed in parallel.
vp9_filter_block1d16_v8 was also optimized in the same way also some of the
loads were being done outside of the loop and by that preventing redundant
loads.
This Optimization gives 43% function level gain and 1.3% user level gain.
Now can be compiled in Windows

Change-Id: I2714124cfb0c14a77d7a0ce126a20db92ffbf92c

Flushing the encoder in simple_decoder example.

Change-Id: I39d78facc79a364cdc529b8cc4930d74f766d413

Adding and reusing vpx_img_plane_{width, height}() functions.

Change-Id: I668afb50614644d2eac2e968e1fafd6d73170bec

Merge "Removing init_rdcost_stack() + cleanup."

Merge "decode_test_driver: Use size_t for size arg to DecodeFrame()."

Merge "Converting int_mv to MV."

Merge "Remove unnecessary vp9_setup_buffer_inter use"

Merge "Remove inactive control parameters"

Merge "Remove unused using_small_partition_info functions"

Merge "Remove unused function in vp9_encodeframe"

Merge "Adding consts to mv search function arguments."

Merge "gen_msvs_vcxproj.sh: Add /arch:AVX for files using AVX."

vp9_rdopt.c static analysis issues resolved

Change-Id: Id53b574a4ecb5c8749383b3758c46b6bc1b275d5

decode_test_driver: Use size_t for size arg to DecodeFrame().

Use size_t for DecodeFrame()'s size arg, and cast only
at the vpx_codec_decode() call site. This silences warnings that
appear in svc_test.cc when building with vs2013.

Change-Id: I2cf39f02a45732c752097f07b0c7ad414b1517d8

Converting int_mv to MV.

Change-Id: Id15285aa48ac6d8fec19a1946e6391412d84f5be

Merge "Cleanup tokenize_b and hoist some invarients out of the loop."

Merge "Using video writer API from vp9_spatial_scalable_encoder."

Merge "Adding API to get vpx encoder/decoder interface."

Remove unnecessary vp9_setup_buffer_inter use

This function initializes the predictor buffer pointers and
calculates reference motion vectors. It is only needed in the settings
of inter frame coding. Hence removing it from the key frame coding
branch in rtc_use_partition.

Change-Id: Ic4e16c7467a5f32be4e0bf619ef9d57afb4a7075

Merge "Fix frames that have no references to them."

Adding consts to mv search function arguments.

Change-Id: Ie79114bba4f0cea55d9f701e20d2be2017630f3b

gen_msvs_vcxproj.sh: Add /arch:AVX for files using AVX.

Turns on AVX when the final characters of .c and .cc file names preceding the
.c and .cc file extension contain the substrings avx or avx2. This silences
many MSVC warnings issued during compilation files that use AVX.

Change-Id: I82bda394af7a688679abab2a50dd7e10b3cb0c7a

Remove inactive control parameters

Change-Id: Ic5692af975fe6bd2d8ec82bbae103c6f7c2fc13e

Remove unused using_small_partition_info functions

This function is deprecated after the re-design of partition search
that runs big block size, then four-way split, followed by
rectangular block sizes. This commit removes the related functions.

Change-Id: I417549c8e0fa3cf35bd29816b805dd4e7c3660c6

Remove unused function in vp9_encodeframe

The function rd_pick_reference_frame can be deprecated. Its use was
subsumed by the adaptive motion search control.

Change-Id: Icb0c2fa335f0f06fa7b79a71f972d9fa54d750db

Merge "Cleaning up vp9_full_search_sad_c() function."

Merge "Removing vp9_foreach_transformed_block_uv() function."

static analysis warning in vp9_ratectrl.c

Change-Id: I8aa3dd068d9a7225b3e8b24392a7b4a54b9435dd

vp9_mcomp.c static analysis issues

Change-Id: I8c0023e3f03297993431e21158ee3821af75f82d

clear out static analysis warning

Change-Id: Ic8708bb8a56ab30617c8ef47072b53928a53a1a3

clean out static analyzer warnings vp9_encodeframe.c

Change-Id: If14d2fc8cf4202d24f7d87ea74b8ce88a6a816d9

Using video writer API from vp9_spatial_scalable_encoder.

Change-Id: I6c2748bef22cdbbc9574d7a8dfc1ed55f4f6b05f

Merge "Minor cleanup."

Merge "Use INTER_OFFSET in vp9_pick_inter_mode"

Merge "Fixed a bug where no valid partition is allowed"

Merge "Skip COMDAT sections when parsing COFF"

Merge "Removes active_worst_quality feedback"

Minor cleanup.

Change-Id: I6ef0ce5371ed9e2a415dd8c08c93ad603f0ad7b2

Use INTER_OFFSET in vp9_pick_inter_mode

Cosmetic change to use pre-defined macros.

Change-Id: I93e9fa90113d0242599048940b39694660385a6f

Fixed a bug where no valid partition is allowed

Change-Id: I4d2729dc5c46db2847700256941a66b0957c105d

Adding API to get vpx encoder/decoder interface.

Change-Id: I137e5e6585356792913e1e84da6c0a439c5153a5

Merge "Enable encode_breakout in real time encoding"

Fix frames that have no references to them.

BasicRateTargeting3TemporalLayers found an error with
the get/release frame buffer management.

Change-Id: Iccff25cba1d2f276f2a0d0fb7b20cf99b4405549

Removing init_rdcost_stack() + cleanup.

Change-Id: I6fcbc833063c658749b2d43bb9968a5762647061

Removing vp9_foreach_transformed_block_uv() function.

Change-Id: I35ec77b71e6fd686865cead9281e4dd9e9bc9e86

Merge "Hiding encode_b_args struct inside vp9_encodemb.c."

Cleaning up vp9_full_search_sad_c() function.

Change-Id: Ief941ff75c5e25f3288326a162b1bd0c0f17a779

Removes active_worst_quality feedback

Removes certain cases of feedback of active_worst_quality,
and removes it from the RATE_CONTROL structure. Now active
worst quality is expected to be computed locally in the
q picking function during the encode.
Making temporal filter strength depend on avg_frame_qindex
rather than on active_worst_quality actually improves
performance esp. for yt.
derf: +0.038%
yt: +0.359%

Change-Id: I1fe5a343034b55af9322289165321f00ac0827b1

Merge "vpxenc: Silence MSVC warnings."

Enable encode_breakout in real time encoding

In real time encoding, we enable encode_breakout to make encoding
fast. A speed feature "use_encode_breakout" is defined to set
encode_breakout thresholds for different speeds.

However, currently, static_thresh is an encoder option. The encode_
breakout can be turned off if user sets static_thresh=0 specifically.

The rtc set borg test result: (need to set --static_thresh=1)
speed -5, psnr loss -3.543%;
speed -4, psnr loss -2.358%;
speed -3, psnr loss -0.771%.

Encoding speed test:
speed -5, 11% - 60% speedup;
speed -4, 5.5% - 28% speedup;
speed -3, 0.8% - 7% speedup.

Change-Id: Icde592ffbe77eac7446f872a2e9eb2051733677b

Merge "Disabling BasicRateTargeting3TemporalLayers test."

Merge "vp9/common/x86: Silence MSVC warnings in vp9_asm_stubs.c."

Disabling BasicRateTargeting3TemporalLayers test.

Frank is going to fix it soon, disabling for now.

Change-Id: Ifb7092d7db5af01bde62897af189fd620b7bbdf3

Merge "Cleaning up compute_fast_motion_search_level()."

Merge "Fixing errorperbit calculation."

vpxenc: Silence MSVC warnings.

Change-Id: I7dd8532052f0b2d4077f579e78db3987b58e3d2a

Merge "vpx_temporal_scalable_patterns: Remove extra const."

Merge "Rework mode_info update in vp9_pick_inter_mode"

Merge "Use more meaningful names for speed features"

Modified Aq1 and Aq2

Aq 1 only updates segment map on kf and arf and
only uses 3 segments. With these settings AQ1 is
+ for most clips in SSIM but negative in psnr.
However, the penalty in PSNR is much less than
previously.

Old version aq1 average results for std hd
-20.899% psnr, -5.809% SSIM
New version aq1 for std hd
-3.57% psnr, +1.23% SSIM

Aq2 Now uses only 2 segments and rd.
This mode is still slightly negative for most clips on
psnr and SSIM but seems to have a much bigger visual
impact on several problem clips than aq mode 1.

Old results for std hd:
-2.578% psnr, -1.151% SSIM
New results for std hd:
-1.561% psnr, -0.85% SSIM

Change-Id: I94f57f8a73121629ce598fb921aad761c1450e1c

Merge "Add get release decoder frame buffer functions."

Merge "Encoder quantization cleanup."

vpx_temporal_scalable_patterns: Remove extra const.

Silences MSVC warning.

Change-Id: I43191cdd8f0231b259ec091be24325ec99d2e962

Merge "Cleaning up update_stats() function."

Merge "Making vp9_activity_masking() static."

Merge "vpx_integer: Use inttypes.h in MSVC (2013 and later)."

Merge "Further one-pass vbr rate control changes"

Merge "*.mk: s/\bUSE_X86INC/CONFIG_USE_X86INC/"

vpx_integer: Use inttypes.h in MSVC (2013 and later).

Change-Id: I2c47f0ab13de82de8833c3f134e5f3c74394374f

Rework mode_info update in vp9_pick_inter_mode

This commit moves unnecessary mode_info update steps out of inter
mode test loop.

Change-Id: I3705450c44cda060021f4fd76c34f1b20d11658b

Use more meaningful names for speed features

Use frame_parameter_update to precisely describe the functionality.

Change-Id: Ia9a55ba8efef7b987e30d949dd00ac716189bdb9

vp9/common/x86: Silence MSVC warnings in vp9_asm_stubs.c.

Update filter_1dfunction definition to match usage.

Change-Id: Ie3cae13dc1ec3f5838c5f29d1c76a1a98a9217fa

Merge "Merged two similar functions to reduce duplication"

Merge "Cleaning up {simple, twopass}_encoder examples."

Further one-pass vbr rate control changes

Some parameter changes and fixes on one-pass rate control.
derfraw300 is now only 10% below 2-pass speed 0 rate control.

Change-Id: I1940eef8a5a035dc18e71b880d5e00cabd1f01b9

Merge "Cleaning up rd_check_segment_txsize() function."

Add get release decoder frame buffer functions.

This CL changes libvpx to call a function when a frame buffer
is needed for decode. Libvpx will call a release callback when
no other frames reference the frame buffer. This CL adds a
default implementation of the frame buffer callbacks. Currently
only VP9 is supported. A future CL will add support for
applications to supply their own frame buffer callbacks.

Change-Id: I1405a320118f1cdd95f80c670d52b085a62cb10d

Merge "add_warning(): remove dead store"

Merged two similar functions to reduce duplication

Function encode_rtc_frame_internal() and encode_frame_internal() only
differed by a couple of speed features, this commit relocation those
difference into the setup of speed features and merged two functions
into one to remove duplication.

It also fixed a subtle bug super_fast_rtc was used before it was
initialized.

Change-Id: I234a5a1d11a4450930e5b4943dbab434208d5030

Merge "Convert small static header functions to inline"

Merge "Convert small static functions in header to inline.."

Merge "Convert small header functions to inline"

Merge "Convert header static functions to inline or make them global."

Merge "Converted functions in header to INLINE..."

Merge "Convert functions to inline that are small ."