Merge "Removing two unused TX_SIZE_SEARCH_METHOD members."

Merge changes I6abc0657,I8224fba2,I04f64a45,I5d49d119,I76b4d171,I88c11ac3

* changes:
  vp9_sub_pixel_*variance*: disable avx2 variants
  vp9_sad*x4d: disable avx2 variants
  vp9_f(dct|ht): disable avx2 variants
  convolve: disable avx2 variants
  fdct8x8_test: add missing avx2 functions
  dct4x4_test: add missing avx2 functions

vp9_sub_pixel_*variance*: disable avx2 variants

tests failing under Win32/Win64

+ variance_test: add missing avx2 functions (partially disabled)

Change-Id: I6abc0657ea076379ab9ca65c12678b9ea199849d

vp9_sad*x4d: disable avx2 variants

tests failing under Win32/Win64

+ sad_test: add missing avx2 functions (disabled)

Change-Id: I8224fba2b270f6039ab1877d71e1e512f0081856

Merge "Makefile: skip .d inclusion for 'testdata' target"

Merge "Add mode info arrays and mode info index."

Add mode info arrays and mode info index.

In non frame-parallel decoding, this works the same way as
current decoding scheme. Every time after decoder finish
decoding a frame, it will swap the current mode info pointer
and  previous mode info pointer if the decoded frame needs
to be shown. Both mode info pointer and previous mode info
pointer are from mode info arrays.

In frame-parallel decoding, this will become more complicated
as current frame's mode info pointer will be shared with next
frame as previous mode info pointer. But when one decoder
thread finishes decoding one frame and starts to work on next
available frame, it needs to retain the decoded frame's mode
info pointers until next frame finishes decoding. The mode info
index will serve this purpose. The decoder will use different
buffer in the mode info arrays and use the other buffer to save
previous decoded frame’s mode info.

Change-Id: If11d57d8eb0ee38c8876158e5482177fcb229428

Removing two unused TX_SIZE_SEARCH_METHOD members.

Change-Id: I33a38bb9f46e7ef509bbbf0cfd7bc3ea5072d022

Merge "Removing chessboard_index from SPEED_FEATURES."

Merge "Removing unused motion_vector_context enum from vp9_encodeframe.c"

Merge "Add x86_64-iphonesimulator-gcc target."

vp9_f(dct|ht): disable avx2 variants

tests failing under Win32/Win64

+ dct16x16_test: add missing avx2 functions (partially disabled)

exercises the forward transforms
no idct/iht implementations, so the c-code is used

Change-Id: I04f64a457fa0828a00f32b5c9fe4f55294f21f61

convolve: disable avx2 variants

tests failing under Win32/Win64

Change-Id: I5d49d11911bcda3a832b14efe5500d22597bedcf

fdct8x8_test: add missing avx2 functions

exercises the forward transforms
no idct/iht implementations, so the c-code is used

Change-Id: I76b4d1712f10225c1ffa5ffb0ed9a551e68b93b4

dct4x4_test: add missing avx2 functions

exercises the forward transforms
no idct/iht implementations, so the c-code is used

Change-Id: I88c11ac37ac6456de9b4b3ec5da059faf4ad6066

Merge "Use small transform size in non-rd real-time mode"

Merge "Removing unused tt_activity_measure()."

Use small transform size in non-rd real-time mode

In non-rd real-time mode, choosing smaller transform size in
encoding gives better video quality and good speed gain than
choosing larger transform size. This patch set tx size search
method to ALLOW_8X8, which is better than using 4x4 or other
larger sizes.

Borg tests on rtc set at speed 6 showed significant gain on quality.
PSNR gain: 11.034% and SSIM gain: 15.466%.

The speed gain is 5% - 12% for <720p clips, and 2% - 7% for
720p clips.

Change-Id: If4dc74ed2df359346b059f47fb73b4a0193ec548

Merge "Fix internal stats printing"

Makefile: skip .d inclusion for 'testdata' target

avoids some unnecessary work when pulling testdata in a freshly
configured tree

Change-Id: Ib43379b0e1549107f3f1b227afca49a899bcc3ab

Merge "gen_msvs_*proj.sh: speed up file generation"

gen_msvs_*proj.sh: speed up file generation

execute fix_path once on the source file list rather than once per entry

Change-Id: Ibc8226e391b3028c1b0bcfeab83c790387c9fe23

Add x86_64-iphonesimulator-gcc target.

macho64 x86_64 with:
- -miphoneos-version-min=5.0
- -isysroot from xcrun --sdk iphonesimulator

Change-Id: I1ba183e16d50c93d48f526e2dc43a7a6040741c5

Merge "Revert "Removing this_frame_stats member from TWO_PASS struct.""

Revert "Removing this_frame_stats member from TWO_PASS struct."

Use of stack frame variable "fps" beyond the lifetime of the function.

fps is sent as a paremeter to output_stats and stored in the
packet holding this encoded frame. This has scope beyond the
lifetime of the calling function.

This reverts commit 3f95a230c7e54104201b3f6c577dfa9251b4642a

Change-Id: Icd8e14b3d7dd733590ada12e619b9dce95b6b0f5

Merge "Adding encode_tiles() function."

Merge "Fixes qindex for first frame in 1-pass cq/q modes"

Merge "README: add a note about the test vectors"

Merge changes I0e4d807f,Ia5ff575c,Ie4a1f313

* changes:
  gen_msvs_*proj.sh: strip SRC_PATH_BARE from obj names
  *.mk: pass SRC_PATH_BARE to all GEN_VCPROJ invocations
  build/msvs: fix builds in source dirs with spaces

Removing chessboard_index from SPEED_FEATURES.

This is not a speed feature, adding inline function instead.

Change-Id: Ia48c41802eec9e92cf990339d724097279695c9a

Adding encode_tiles() function.

Change-Id: Ib8187c8f2556e1e9268b0683cd2b6ff3489f0205

Merge "Neon match to vp8 temporal denoiser fix"

Fixes qindex for first frame in 1-pass cq/q modes

Produces sane qindex for the first frame in 1-pass constant and
constrained qualirty modes.

Change-Id: Ib2a5091df15a23489e9bb5534a2019cf2689755e

Merge changes I59ef7c49,I0156d856

* changes:
  Made MACROBLOCK.increase_denoising cond-compiled
  s/INT_MAX/UINT_MAX/ where appropriate

Fix internal stats printing

Change-Id: I61bd0b127164a591b1c983bfcebd64ba7617f796

Add x86-iphonesimulator-gcc target.

macho32 i386 with:
- -miphoneos-version-min=5.0
- -isysroot from xcrun --sdk iphonesimulator

Change-Id: I6bb43eebab39a71cdb76264644eba14c22d736c2

Removing unused tt_activity_measure().

Change-Id: Ifcb46e6904730d14b9ef76b648b4d0dc3cd5d0c5

Removing unused motion_vector_context enum from vp9_encodeframe.c

The same enum defined and used in vp9_mvref_common.c.

Change-Id: I3975103997797add0a258d36c96d20ac9561a73d

Removing unused alt_freq field from VP9EncoderConfig.

Change-Id: I9b683c8647a864e74073161f4aa6f2911b7825e3

Made MACROBLOCK.increase_denoising cond-compiled

Change-Id: I59ef7c49f72d2d40bbe5b56af11bdf5f9ae2f1b8

s/INT_MAX/UINT_MAX/ where appropriate

Change-Id: I0156d85671305326525c4644510e240021eca461

Merge "Enable unit test for partial 16x16 inverse 2D-DCT"

Using 2 instead of 3 elements for avg_frame_qindex array.

The third array element was unused. 2 elements now: key- and interframe.

Change-Id: I5b8b9f5d889cc96a204cedfc432059293256298e

Enable unit test for partial 16x16 inverse 2D-DCT

This commit enables unit test for SSSE3 16x16 inverse 2D-DCT with
10 non-zero coefficients. It includes a new test condition to
cover the potential overflow issue due to extremely coarse quantization.

Change-Id: I945e16f05dfbe19500f0da5f15990feba8e26d99

Merge "Re-enable unit test for SSSE3 forward 8x8 2D-DCT"

Merge "Enable SSSE3 inverse 2D-DCT with 10 non-zero coeffs"

Re-enable unit test for SSSE3 forward 8x8 2D-DCT

This commit enables the unit test for SSSE3 forward 8x8 2D-DCT.

Change-Id: Ie4aae84ac74f6eb1a80b62285c91632bd6db29f1

Merge "Removing lossless field from VP9EncoderConfig."

Merge "Fix potential overflow issue in SSSE3 forward 8x8 2D-DCT"

Merge "Rework unit test for 8x8 transformation"

Fix potential overflow issue in SSSE3 forward 8x8 2D-DCT

The SSSE3 implementation might find a potential overflow issue in
its second 1-D transform, if all input residual pixels are close to
255. This commit fixes the issue and re-enables the unit test on
the SSSE3 version.

Change-Id: I0520478abdab7afd3ff2842516bec951111e9b3c

Rework unit test for 8x8 transformation

This commit reworks the unit test for 8x8 forward/inverse
transformation. It adds extreme input value test to detect overflow
issues in the intermediate steps.

It temporarily disables unit test for the SSSE3 version, which
showed overflow failure in the new test conditions.

Change-Id: I7caf10bba4b6db031add65d8c0eb99426b38aa42

Adding buffer levels to RATE_CONTROL struct.

Change-Id: Ib35ff854378764dc3c6745844c67a33dee545663

Removing lossless field from VP9EncoderConfig.

Right now there is just one place to check: xd->lossless and for the first
pass there is a function is_lossless_requested().

Change-Id: I949a6834e64ce51e422e2892f097f2b871b5429a

Merge "Cleaning up full_pixel_search()."

Merge "Moving first pass related functions to vp9_firstpasss.c."

Merge "Reusing existing vp9_get{8x8, 16x16}var() instead of new ones."

Fix AQ mode 2 bug where delta causes Q 0.

In Aq mode 2 for kf/arf/gf the segment q delta
is calculated and then applied by re-quantization without
going through the rd loop again. If the base Q != 0
but the segment Q == 0 (lossless) this can could give rise
to a situation where we have an illegal combination of
transform size and Q. (Q == 0 requires that all blocks
are coded 4x4 WHT).

Change-Id: I241a58c6494ed442e9e4630070b0cde0fb99ae45

Merge "Remove Wextra warnings from vp9_sad.c"

Merge "Remove an attempt to handle SEG_LVL_SKIP sub8x8."

gen_msvs_*proj.sh: strip SRC_PATH_BARE from obj names

reduces the risk of exceeding msbuild command line length limits

Change-Id: I0e4d807f42c090a709a3dc365a02b929a3071688

*.mk: pass SRC_PATH_BARE to all GEN_VCPROJ invocations

this will enable stripping of the path from the object file name

Change-Id: Ia5ff575cfa4fe946200878b0aded05832609f316

build/msvs: fix builds in source dirs with spaces

...when configured below the path containing spaces. configuring outside
the path containing spaces still won't work due to issues with the
makefiles, e.g.,
/path with spaces/git
/path with spaces/build1
/build2
configure/make in build1 will work, build2 will not

Change-Id: Ie4a1f313596d7457cadd67476ac1dbd3273ad46e

Remove Wextra warnings from vp9_sad.c

As a side-effect, the sad unit tests for VP8 and VP9
had to be separated.

Fixes a bug in original patch:
(https://gerrit.chromium.org/gerrit/#/c/70163/8)
that was reverted due to a nightly test failure.

Change-Id: Ia2a4e9e278fd3c89d6c3c82fcc6381320ec2a8a6

Fixing failed ARM build.

Change-Id: I3f74418f07c2dfdd7725a5b4a8ef5c5f4aca6289

Merge "Fixed OUTPUT_YUV_SRC behavior for VP8"

Merge "seeing a 10x slowing down, revert now for investigation"

seeing a 10x slowing down, revert now for investigation

Revert "Fix a problem of using an uninitialized parameter"

This reverts commit 538af7db5ffe2ab560511a3bc0ae74dae9316c14

Change-Id: I071aa9b7068ef515abb8ae9584df15067706ccb5

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

Revert "Remove Wextra warnings from vp9_sad.c"

This reverts commit 916550428db803c54c993ff9d3c34b9b0bcebb7c

Change-Id: I500822b03f09c64ff6ec5396c68edee9ca3b75cb

Merge "Converting disable_inter_mode_mask to inter_mode_mask."

README: add a note about the test vectors

Change-Id: I8b3299bf1579962c6dbb29b61c16801fef2460fd

Merge "Add overflow check unit test for 16x16 inverse DCT/ADST transform"

Merge "Fix a potential overflow issue in inverse 16x16 full 2D-DCT"

Merge "Fix unused variable warning."

Merge "Refactor the vp9_get_frame code for frame parallel."

Add overflow check unit test for 16x16 inverse DCT/ADST transform

This commit applies quantization process with coarse quantization
step size to the forward transform coefficients and tests all the
inverse 16x16 DCT and ADST implementation versions with the
dequantized coefficients as input, to verify that the outcomes
match the prototype.

Change-Id: I68034a6126b45192c87d8c642155290e89bff8fa

Merge "Fix a problem of using an uninitialized parameter"

Merge "Removing unused ref_frame_mask local var."

Refactor the vp9_get_frame code for frame parallel.

In frame parallel decoding mode,  there will be still several frames inside
the decoder when application stop calling vpx_codec_decode to decode frames.
The application will need to keep calling vpx_codec_get_frame to get all the
remaining decoded frames in the decoder.

Change-Id: I2ce8260a91282f045bb9a6093ff8a606b1990f14

Merge "Do not export non-existant symbols"

Fix unused variable warning.

Change-Id: I52f73943194039697da6880d2da54821b2498336

Fix a problem of using an uninitialized parameter

This commit added a call to set speed feature before initializing
motion search, fixed the problem where unintialized search method
is used before its value being set.

Change-Id: I537e4612bf0d00fd6f51396fd222d4b3bd6fde58

Fixed OUTPUT_YUV_SRC behavior for VP8

By enabling the OUTPUT_YUV_SRC compiler flag, the encoder will write the raw
input to bd.yuv.

The functionality was mostly implemented, but in its previous state did not
compile.

Change-Id: Ia331ad0f4c6e6f9f51e8d42cd33ba8cc146b3dbf

Merge "Re-factor some duplicate code."

Do not export non-existant symbols

Change-Id: Ic0fb8e5016d064c2227cf5d87ffba86a4303c82b

Removing unused ref_frame_mask local var.

Change-Id: Ie11558c076a0161cc9608788e050b1b16e31c490

Merge "Consistent names for intra mask flags."

Merge "Don't update encoder skip count for SEG_LVL_SKIP."

Converting disable_inter_mode_mask to inter_mode_mask.

Making this consistent with intra mode masks: you need to specify
allowed inter/intra modes to use.

Change-Id: Iaecd28bf79047259707d8e7a59a57bb7b856383e

Consistent names for intra mask flags.

Change-Id: Ibdd5255d37200fb8a1d50f71a2a49c6089ae21e7

Remove an attempt to handle SEG_LVL_SKIP sub8x8.

SEG_LEVEL_SKIP requires the block size to be at least 8x8. Attempting to
use it on smaller partitions causes the decoder to reject the bitstream.

Change-Id: Ia7188cdf8ae5ac1df6bd29f3f80dbb0610e1f7b1

Merge "Making speed checks consistent in set_rt_speed_feature()."

Fix a potential overflow issue in inverse 16x16 full 2D-DCT

An overflow issue could potentially happen in the second round 1-D
transform of the SSSE3 full inverse 16x16 2D-DCT. This commit fixes
this issue.

Change-Id: Ia19e4888fda1cc929a28a5f89a5beec612d628dc

Don't update encoder skip count for SEG_LVL_SKIP.

This aligns the encoder behavior with the decoder.

Change-Id: Ifa0840e4b07b19309e0bf1d1182498883249ec45

Merge "Moving itxm_add pointer from MACROBLOCKD to MACROBLOCK."

Reusing existing vp9_get{8x8, 16x16}var() instead of new ones.

Change-Id: I87b7c657d8813d7fb383ab519d150c0ffb1dd377

Making speed checks consistent in set_rt_speed_feature().

Change-Id: Id3d0a49836fe996b806707d29a8130acf9d7ea0e

Merge "Fixing -Wextra warnings in vp9_{cx, dx}_iface.c."

Merge "Fixed a crash windows build"