Merge "Revert "Revert "VP8 for ARMv8 by using NEON intrinsics 08"""

Merge "Removing lookup_next_frame_stats()."

Merge "Skip intra mode tests depending on inter residuals"

Merge "Replacing asm 8x8 variance calculation with intrinsics."

Removing dummy_packing member from VP9_COMP.

Change-Id: I571ce84c97087f8a1a36a10058393bfdcefbf72a

Replacing asm 8x8 variance calculation with intrinsics.

New code is 10% faster for 64-bit and 25% faster for 32-bit. Compiled
using clang.

Change-Id: I8ba1544c30dd6f3ca479db806384317549650dfc

Merge "Fix int64_t to unsigned int conversion warnings"

Merge "Extend block level sse to support multiple txfm blocks"

Fix int64_t to unsigned int conversion warnings

Use unsigned int type to store the sse in the pixel domain. The
precision is sufficient to handle sse of block size up to 64x64.
The transform domain version however needs int64_t, since there is
a transfer gain applied in the forward transformation that might
cause unsigned int overflow.

Change-Id: Ifef97c38597e426262290f35341fbb093cf0a079

Merge "Removing variance MMX code."

Merge "vp9: sync workers at the start of decode_tiles_mt()"

Merge "vp9: fix m/t loop filter invalid free"

Merge "Minor fix in vp9_encoder.h"

Revert "Revert "VP8 for ARMv8 by using NEON intrinsics 08""

This reverts commit 928ff03889dadc3f63883553b443c08e625b4885

Compiles with 4.6 now.

Change-Id: Ib455da1098bb0e0623248be07579882a425fcbd1

Minor fix in vp9_encoder.h

Added the missing "int".

Change-Id: I7c8af3dee700837b40f010d53e1431a59370ae3a

vp9: fix m/t loop filter invalid free

store the number of allocated rows in VP9LfSync, the calculated values
can not be relied on when dealing with corrupt material.

Change-Id: I13b8bcec9738c299a71df726772ab7ac05511e5b

Removing variance MMX code.

Removed functions:
* vp9_mse16x16_mmx
* vp9_get_mb_ss_mmx
* vp9_get4x4var_mmx
* vp9_get8x8var_mmx
* vp9_variance4x4_mmx
* vp9_variance8x8_mmx
* vp9_variance16x16_mmx
* vp9_variance16x8_mmx
* vp9_variance8x16_mmx

They all have SSE2 equivalent.

Change-Id: I3796f2477c4f59b35b4828f46a300c16e62a2615

Skip intra mode tests depending on inter residuals

This commit allows encoder to skip intra coding mode test, when
the known inter residual is less than the source variance. It
reduces the runtime of speed 3 for test clips:
bus cif 1000 kbps: 8587 ms -> 8260 ms, 3.8% speed-up
pedestrian 1080p 2000 kbps: 161381 ms -> 155241 ms, 3.7% speed-up.

The compression performance is down by
derf   -0.36%
stdhd  -0.25%

Change-Id: I75ce1e035b4da2153cb1ac14111d1a07c05a735d

Extend block level sse to support multiple txfm blocks

This commit extends the sse and forward transform computation flag
to support the case 64x64 blocks where there are 4 32x32 2D-DCT
blocks.

Change-Id: I86a3e805dfaa0f3abd812f590520c71aa0e40473

Merge "vp9: fix crash in inline loopfilter w/corrupt file"

Merge "vp9: fix crash in mt loopfilter w/corrupt file"

vp9: sync workers at the start of decode_tiles_mt()

prevents any problems resuming decode after decoding a corrupt frame

Change-Id: Ib7eb1b5c062aebe71074fef1ece32a32822c16be

Merge "Implementing 4x4 variance calculation with SSE2."

Implementing 4x4 variance calculation with SSE2.

New SSE2 function is three times faster than MMX one.

Change-Id: I4f387ce9f75b88379176ec7bdc62d86eb5f70fbe

Removing alg_priv from vpx_codec_priv struct.

In order to understand memory layout consider the declaration of the
following structs. The first one is a part of our API:

struct vpx_codec_ctx {
  // ...
  struct vpx_codec_priv *priv;
};

The second one is defined in vpx_codec_internal.h:

struct vpx_codec_priv {
  // ...
};

The following struct is defined 4 times for encoder/decoder VP8/VP9:

struct vpx_codec_alg_priv {
  struct vpx_codec_priv base;
  // ...
};

Private data allocation for the given ctx:

struct vpx_codec_ctx *ctx = <get>
struct vpx_codec_alg_priv *alg_priv = <allocate>
ctx->priv = (struct vpx_codec_priv *)alg_priv;

The cast works because vpx_codec_alg_priv has a
vpx_codec_priv instance as a first member 'base'.

Change-Id: I10d1afc8c9a7dfda50baade8c7b0296678bdb0d0

Merge "Removing unused arnr_type from VP9EncoderConfig and vp9_extracfg."

Merge "Early termination in encoding partition search"

Merge "vp8_multi_resolution_encoder: Huge cleanup."

Merge "test/vp8_multi_resolution_encoder.sh: Add missing devnull redirect."

Merge "Removing unused debug code under WRITE_RECON_BUFFER."

Merge "Converting configure_skippable_frame() to is_skippable_frame()."

Merge "Renaming vpxenc option from frame_boost to frame-boost."

Merge "Adding several vpxenc parameter ranges to help output."

Early termination in encoding partition search

In the partition search, the encoder checks all possible
partitionings in the superblock's partition search tree.
This patch proposed a set of criteria for partition search
early termination, which effectively decided whether or
not to terminate the search in current branch based on the
"skippable" result of the quantized transform coefficients.
The "skippable" information was gathered during the
partition mode search, and no overhead calculations were
introduced.

This patch gives significant encoding speed gains without
sacrificing the quality.

Borg test results:
1. At speed 1,
   stdhd set: psnr: +0.074%, ssim: +0.093%;
   derf set:  psnr: -0.024%, ssim: +0.011%;
2. At speed 2,
   stdhd set: psnr: +0.033%, ssim: +0.100%;
   derf set:  psnr: -0.062%, ssim: +0.003%;
3. At speed 3,
   stdhd set: psnr: +0.060%, ssim: +0.190%;
   derf set:  psnr: -0.064%, ssim: -0.002%;
4. At speed 4,
   stdhd set: psnr: +0.070%, ssim: +0.143%;
   derf set:  psnr: -0.104%, ssim: +0.039%;

The speedup ranges from several percent to 60+%.
                 speed1    speed2    speed3    speed4
(1080p, 100f):
old_town_cross:  48.2%     23.9%     20.8%     16.5%
park_joy:        11.4%     17.8%     29.4%     18.2%
pedestrian_area: 10.7%      4.0%      4.2%      2.4%
(720p, 200f):
mobcal:          68.1%     36.3%     34.4%     17.7%
parkrun:         15.8%     24.2%     37.1%     16.8%
shields:         45.1%     32.8%     30.1%      9.6%
(cif, 300f)
bus:              3.7%     10.4%     14.0%      7.9%
deadline:        13.6%     14.8%     12.6%     10.9%
mobile:           5.3%     11.5%     14.7%     10.7%

Change-Id: I246c38fb952ad762ce5e365711235b605f470a66

vp8_multi_resolution_encoder: Huge cleanup.

Change-Id: I65b2c1fbed5a306949843315999d10368a100431

test/vp8_multi_resolution_encoder.sh: Add missing devnull redirect.

Change-Id: Id60ae8178191e30603a5c8665793ace7eb9c8533

Adding several vpxenc parameter ranges to help output.

Change-Id: Iec1a00203930a539ed584961aed4b2bfd7488255

Merge "Updates vp9_pattern search to return integer sads"

Merge "Replacing int_mv with MV."

Updates vp9_pattern search to return integer sads

Updates the vp9_pattern_search function to return integer one-away
neighbors' sad values, for subsequent use in speeding up the
sub-pel search. Also, removes code for the do_refine option
which is not being used currently.
Updates the integer and subpel functions to pass in a 5-element
sad list for output or input.

A new pruned sub-pel search algorithm is implemented that uses
the sad returned from the integer pel search. But it is not
deployed yet.

Change-Id: Ifa9f5ad024b5b660570366d2bd900343e1891520

Merge "Re-work RD modeling based on inter frame prediction residual"

Merge "Re-use switchable rate value in handle_inter_mode"

Merge "Add an early termination check in handle_inter_mode"

Merge "Use max txfm size unit in rate-distortion cost modeling"

vp9: fix crash in inline loopfilter w/corrupt file

attempting to decode a frame after the previous frame failed has the
potential of interrupting an earlier loop filter task

Change-Id: I6f2b1ddcdf5b89c3e2ee8caf5289dada2a087d66

Add test for vp8_multi_resolution_encoder.

Change-Id: I94667a23046e3cf17c421ac97ca66ea57f846b5c

Re-work RD modeling based on inter frame prediction residual

This commit re-work the operation flow related to prediction
residual generation and the rate-distortion modeling. It saves one
call for model_rd_for_sb.

Change-Id: Icaf96c0ff09c903637ed5283448afe01d798195f

Re-use switchable rate value in handle_inter_mode

The value of switchable rate has been stored in a local variable.
This change skips the second call to vp9_get_switchable_rate() by
reusing the local variable.

Change-Id: Ib7d3fef7621cc4bde94c6d6e6b3a71f1fd4559f2

Add an early termination check in handle_inter_mode

Check the mode and motion vector cost. If it is already above
the existing best rate-distortion cost, skip the rest check process
on this mode.

Change-Id: Ie065cebdfda2a3be3be18b8e8b43dc29aaa8c179

Use max txfm size unit in rate-distortion cost modeling

This commit makes the rate distortion modeling run in the unit of
maximum transform block size. No compression/speed change observed.
It is for the use of later fast forward transform purpose.

Change-Id: Ibaaedb69c765e8d0c5d5012f0ec07f36fd9f68fd

vp9: fix crash in mt loopfilter w/corrupt file

if the first frame was corrupt and loop filter not called, the next call
would assume the necessary allocations had been done and segfault when
accessing a NULL pointer

Change-Id: Ib6ef505e5c594e6f0fe65ab0700172bcf06b92a6

Add a separate datarate test for denoiser enabled.

Change-Id: Ibc91a46d11ad9c58e340b4346ed813a1be9b404a

Merge "add a new interp filter search strategy."

Merge "Enable denoiser for vp8 datarate tests."

Enable denoiser for vp8 datarate tests.

Keep one test, BasicBufferModel, with denoiser set off.

Change-Id: I762c2004b3adca24f677535cfa5b991108f31f01

Merge "gen_msvs_(vcx)proj.sh: set yasm format explicitly"

Merge "vp8 fix: deallocate denoiser->yv12_last_source"

Replacing int_mv with MV.

Change-Id: I483a2fefc5f9ea4533dfd64448f3b6b426dd9eed

Merge "Set scaled img format correctly in vpxdec"

Merge "twopass_encoder: Flush encoder."

Merge "vpxenc.sh: Move some frequently used params into functions."

vp8 fix: deallocate denoiser->yv12_last_source

Change-Id: I7f3bc292b1b411e10586f0975143ae996ea78d1d

add a new interp filter search strategy.

This commit addes a new strategy to reduce the search for optimal
interpolation filter type. The encoder counts and store how many each
filter type is selected and used for each of the reference frames.
A filter type that is rarely used for all three reference frames is
masked out to avoid computation.

The impact on compression is neglectible:
-0.02% on derf
+0.02% on stdhd

Encoding time is seen to reduce by 2~3%.

Change-Id: Ibafa92291b51185de40da513716222db4b230383

Removing unused arnr_type from VP9EncoderConfig and vp9_extracfg.

Change-Id: Icab9a4399c5687453f4bec14b8cb5000464335e5

twopass_encoder: Flush encoder.

Change-Id: Ib04a545b43f3a36c6df7eaf3ed2d802cf41119bb

Merge "Removing non-executable code from encoder_encode()."

Merge "Removing tx_stepdown_count from VP9_COMP."

Merge "Adding oxcf temp variable."

Merge "Adding is_keyframe temp var."

Renaming vpxenc option from frame_boost to frame-boost.

We use dashes, no underscores.

Change-Id: I423eee946d10731856e15591eb6e9e4628e3ee6c

Removing non-executable code from encoder_encode().

We don't output invisible frames with VP9.

Change-Id: I7b874d3ac454c1b2966d5d7d72e12a864b49afae

Merge "Passing const cfg pointer to vpx_codec_{dec,enc}_init_ver()."

vpxenc.sh: Move some frequently used params into functions.

Echo args back from new util functions to decrease the repetition
in this file.

Change-Id: Id4d7c02bf81cf2c5261f58b560e033869a346677

Removing tx_stepdown_count from VP9_COMP.

The variable is never read.

Change-Id: I94141c1667fa5d10604cd6f83c5f64df107dee94

Merge "[spatial svc]Multiple frame context feature"

Merge "vpxenc.sh: Make vpxenc() usage agree with implementation."

Merge "twopass_encoder: Separate functions for first and second pass."

vpxenc.sh: Make vpxenc() usage agree with implementation.

- Make $1 input file at all call sites.
- Actually use vpxenc_pipe().

Change-Id: I22f25719aeb3dce60863207b2e2a49e42e27ed06

Merge "vpxenc.sh: Add real time tests."

Passing const cfg pointer to vpx_codec_{dec,enc}_init_ver().

Change-Id: I3e319f581215ccb814afca247c1d911be231a4ac

Merge "Replacing int_mv with MV inside the first pass code."

Merge "Cleaning up is_background()."

twopass_encoder: Separate functions for first and second pass.

Change-Id: Ic5e3ff5f572ed55d7dd3df8c606e57daacf885dc

Merge "Removing source field from VP9_COMP."

Merge "Removing is_best_mode() function."

Cleaning up is_background().

Change-Id: I2b9609dd22bacbf26e669f70bf155613b0316eb3

[spatial svc]Multiple frame context feature

We can use one frame context for each layer so that we don't have
to reset the probs every frame. But we can't use prev_mi since we
may drop enhancement layers. So we have to generate a non vp9
compatible bitstream and modify it in the player.
1. We need to code all frames as invisible frame to let prev_mi
   not to be used. But in the bitstream we need to code the
   show_frame flag to 1 so that the publisher will know it's
   supposed to be a visible frame.
2. In the player we need to change the show_frame flag to 0 for
   all frames. Then add an one byte frame into the super frame
   to tell the decoder which layer we want to show.
Change-Id: I75b7304cf31f0ab952f043e33c034495e88f01f3

Merge "enable wextra in vp9"

Merge "vp9_calc_ssim: remove unused parm to resolve warning"

Merge "vp9_denoiser_update_frame_stats: unused parm fixed"

Merge "vp9_postproc.c: unused parameter warning resolved"

invalid_file_test: correct HandlePeekResult signature

restore const lost in:
f0f9ab6 invalid_file_test.cc: remove unused param warnings

Change-Id: I0c5318f84c81e3dd7bb44468463faf9edd6bc18a

Merge "vpxdec: fix --keep-going with --disable-vp8"

Merge "add libyuv/scale_neon64.cc"

Merge "active_map_test: use vpx_active_map_t() to initialize vars"

Merge "Choosing GOOD mode by default."

add libyuv/scale_neon64.cc

from r1051, missed in r1060 snapshot update

Change-Id: I3756d8d29fc0dd4c4491efa84b8f8837edde23a7

vpxdec: fix --keep-going with --disable-vp8

the parsing of this flag was mistakenly put in a CONFIG_VP8_DECODER
conditional block in:
95853db vpxdec: add --keep-going option

Change-Id: Ie83ca0399fd3f3d4b0a9d03b7ca5536b310e1f02

Merge "Removing extraconfig_map from vp8_cx_iface.c."

active_map_test: use vpx_active_map_t() to initialize vars

0-initializes [1] and removes a warning for missing initializer fields

[1] http://en.cppreference.com/w/cpp/language/value_initialization

Change-Id: I886fa72c74679b52cdce2af3364172395a5b6a51