Cleaning up vp9_svc_layercontext.c

Change-Id: I7c94b02f621ccc1a738b0d52edf60e9012cfc014

Merge "Cleaning up vp9_mcomp.c."

Merge "Fix bad speed settings interaction."

Merge "Reusing {above, left}_seg_context vars in both encoder and decoder."

Merge "Removing source & source_sz from VP9Decompressor struct."

Merge "changed to use correct pred_mv"

Merge "Fix a valgrind error"

changed to use correct pred_mv

The third pred_mv is stored in x->pred_mv[ref_frame]. This commit make
sure the correct mv is read.

Change-Id: Ibed24daf36703a63f0394c87b2381ee1d2eb7910

Merge "Remove duplicate declaration"

Fix a valgrind error

Valgrind was complaining uninitialzed values are being used.

Change-Id: Ia9eb619f4e67d1fb7af1057d50304f15adc02aab

Fix bad speed settings interaction.

The flag x->skip_recode interacts badly with
the cpi->sf.use_nonrd_pick_mode and
cpi->sf.skip_encode_sb speed settings.

Restricting the use of the skip_decode flag when
these other speed choices are in use helps quality
for speeds 3 and 4 by a large amount with only a
small impact on speed.

Average improvmentes for 2 pass speed 4:
Derf +8.8%
Yt + 10.53%
Std-Hd +6.95%
yt-hd + 22.95%

Change-Id: I8010876d8012042a11077c92e69d813c3dfa58eb

Merge "Some code cleanup for cyclic refresh."

Change the min/max q validation in lossless mode

This commit changed how q is validated in lossless mode. With this
commit, when --lossless=1 is specificed at commandline, --min-q and
--max-q are now ignored. This is to make the option non-ambiguious.

Change-Id: I33e85690460537509d33be75d6a3597be4affc09

Use local vairable in rd_auto_partition_range()

In addition to a few cleanups.

Change-Id: Ice5938ef494513921a47e7c64ba9928f2202e24e

Disable speed 6 for datarate test.

One of the tests for real-time mode is failing at speed 6.
Introduced recently, will enable again when fixed.

Change-Id: I8f42de6a3eca226c9aa5c5e1fab98d629993c087

Some code cleanup for cyclic refresh.

Change-Id: I0e4e2462ee27640f9bf4091431241ede6fc97267

Reusing {above, left}_seg_context vars in both encoder and decoder.

Change-Id: Id1fa36c92cb007b73a450cc8552e810cedad38b9

Merge "intrapred_test: fix inheritance"

Merge "[svc] Finalize spatial svc first pass rate control"

Removing source & source_sz from VP9Decompressor struct.

Change-Id: If4c1a48c60e8b27dd021a83170289ccc5467c6de

Merge "Removing mi_stream."

Log encoding time/fps in vpx_temporal_scalable_patterns

Change-Id: I368750c3641c5daaa8871f3880af5cfcf5e79bba

Remove unused mode_sad

Removed mode_sad.

Change-Id: I230b42ac9b617ae2c375e297057aa0756bd355fe

ads2armasm_ms: Match the indentation of the previous line

Instead of hardcoding a certain indentation, use the regexp to
provide similar indentation for the new line as well.

Change-Id: Iacb2621b35ce7e1aa3980c1603b8e3ab02d98a35

Merge "configure: test -m(mmx|sse|sse2|sse3) flags"

Merge "ads2armasm_ms: Add an ALIGN 4 after ENDP"

Merge "Enable variable block size test in non-RD mode decision"

Merge "Cleaning up vp9_onyx_if.c and vp9_onyx_int.h."

Merge "Moving common code into vp9_update_mv_count()."

Enable variable block size test in non-RD mode decision

This is an initial attempt to allow variable block size partition
in non-RD coding flow. It tests 8x8, 16x16 and 32x32 block size per
64x64 block, all using non-RD mode decision and the associated rate
distortion costs from modeling, then selects the best block size to
encode the entire 64x64 block. Such operations are triggered every
other 3 frames. The blocks of intermediate frames will reuse the
collocated block's partition type.

It improves the compression performance by 13.2%. Note that the gains
are not evenly distributed. For many hard clips, the compression
performance is improved by 20% to 28%. Local speed test shows that
it will also increase runtime by 50%, as compared to speed -7. It is
now enabled in speed -6 setting.

Change-Id: Ib4fb8659d21621c9075b3c369ddaa9ecb0a4b204

Moving common code into vp9_update_mv_count().

Change-Id: I512482853bdf2695fbdf1c705a2ada354ccf76cf

Adding alloc_mi() function.

Change-Id: I3b944884c048f589c86e0169aeb3c3855bc8b729

ads2armasm_ms: Add an ALIGN 4 after ENDP

This makes sure that labels for data symbols directly after
functions get properly 4-byte-aligned (when the source is assembled
in thumb mode).

Previously, if declaring a data symbol directly after a function, the
symbol could end up pointing to the unaligned address (if the total
size of the thumb function didn't end up being a multiple of 4). The
data in the symbol itself ended up aligned, but the symbol pointed to
the preceding unaligned position.

That is, a source file looking like this:
---
   ...
   ENDP

symbol
   DCD 0x12345678
---

could end up being assembled into
symbol:
  xxxxx2: 0000
  xxxxx4: 5678
  xxxxx6: 1234

(This doesn't happen if the symbol label is on the same line as the
DCD directive.)

By adding an ALIGN 4 directly after the ENDP we make sure the symbol
itself gets aligned properly.

This isn't an issue with the original, untranslated arm source,
since it only is built in arm mode where all instructions are 4 byte,
and since the gnu assembler automatically adds the padding before the
symbol even in thumb mode.

Change-Id: Iadbeebd656b0197e423e79a12a7d3ef8859cf445

Remove duplicate declaration

Change-Id: Ic8e52a89e0df816c38cd8ff1b7c53862b9a6dff2

Merge "Fix the use of uninitialized skip flag"

Merge "Rearranging and removing unused defines."

Merge "Fix the md5 mismatch for some scale cases."

[svc] Finalize spatial svc first pass rate control

      1. Save stats for each spatial layer
      2. Add frame buffer management for svc first pass rc
      3. Set default spatial layer to 1
      4. Flush encoder at the end of stream in test app
This only supports spatial svc.
Change-Id: Ia89cfa87bb6394e6c0405b921d86c426d0a0c9ae

Removing mi_stream.

Change-Id: If674140e30c223c88894b983fd22a583efb99dcf

Merge "Removed several unused functions."

Merge "Remove an unused function"

configure: test -m(mmx|sse|sse2|sse3) flags

<=sse2 isn't strictly necessary on x86_64, but this is more consistent
with the rest of the flags and should be harmless

Change-Id: Ice0f1d1c4c7510ee90af2a62dbd3d6508db63487

update gitignore

vp8_set_maps was renamed in
61ecd7f Add VP9 support to the set maps example.

Change-Id: I6c67254b9c437886e88dca21387ae4e86c74c440

Merge changes If529f310,I0f4a8202

* changes:
  configure: test for -mssse3
  configure: factorize gcc machine option checks

intrapred_test: fix inheritance

inheritance should be public; also correct placement of ClearSystemState
as the base class doesn't inherit from testing

Change-Id: I0f41330fccc62a70b8dd40d66bbd829b9d98cf84

Revert "configure: Check for make"

This reverts commit 89025585cde1e2696c134b3cb7457abd6a3c3933.

This check breaks BSD builds and isn't useful through the configure
process. The README describes the build environment requirements (GNU
make).

Change-Id: I25f8a9c1640909412ab405dbd09a1c4d93e5a511

configure: test for -mssse3

fixes compile with older versions of gcc

Change-Id: If529f3102dbc926be8d5fb91d4161fa686c11840

configure: factorize gcc machine option checks

check_gcc_machine_option() replaces individual -m* checks

Change-Id: I0f4a82020c0541b99209321907e80e071d1245e1

Merge "y4minput: add more error reporting on read failure"

y4minput: add more error reporting on read failure

Change-Id: Iedb136e4019ec3eb3005ea567efb33902dccfb8d

Merge "tokenize: quiet -Warray-bounds warnings"

Merge "rdopt: quiet -Warray-bounds warnings"

Merge "onyx_if: quiet -Warray-bounds warnings"

Merge "svc_encodeframe: quiet -Warray-bounds warnings"

Fix the use of uninitialized skip flag

The use of uninitialized skip flag will trigger inconsistency in
coding statistics, when alternate RD and non-RD coding modes are
enabled. This commit fixes this issue and removes unnecessary if
statements from update_state_rt.

Change-Id: I7d549dcb0e3ef48b999e5bbc78174ba84502cfcf

Merge "Fixing warnings/errors from c++ compiler."

Merge "In-frame q adjustment for cyclic background refresh."

Merge "Cleaning up calc_active_worst_quality_one_pass_vbr()."

Merge "Adding libwebm sources to third_party/libwebm."

Cleaning up vp9_onyx_if.c and vp9_onyx_int.h.

Change-Id: I5136c45d3f5f435005a88d04652addcd835f4bfe

Rearranging and removing unused defines.

Change-Id: I03049f89565ce19a297e65e2d53db127dc6f77d9

In-frame q adjustment for cyclic background refresh.

Activated using aq_mode=3.

Change-Id: Ied628b9e7bd0e88b0c75790276bca75b19eb5c07

Fixing warnings/errors from c++ compiler.

Change-Id: Ia561dda53f2dd10e3a10a2df2adb8027ab19397a

Cleaning up calc_active_worst_quality_one_pass_vbr().

Change-Id: Ibfccee5e35c235c02843e1c1fbd3c39034619cf5

Adding libwebm sources to third_party/libwebm.

Hash and license details are found in README.webm

Change-Id: I52192abe8eba8ec413f6bccd479e8e2256a8f617

Merge "Remove non-420 configure option."

Merge "Add bit-stream write flag to nonrd_use_partition"

Merge "Cleaning up enums in vp9_onyx_int.h"

Merge "Moving defines from vp9_onyx_int.h to suitable *.c files."

Removed several unused functions.

Change-Id: Ib9e27298c575afc02a98b593bc6ad60762064d9b

Merge "Calculate rate_mv after sub-pixel search in non-rd mode"

Remove an unused function

Change-Id: Iaaea882e59dfec38c0a9a24deaf8525a0fb62515

Cleaning up enums in vp9_onyx_int.h

Change-Id: Ib5cbf77052bc03495ae135b3cdcd39c2452c2b34

Moving defines from vp9_onyx_int.h to suitable *.c files.

Change-Id: I0d39bdbb8166ee44561f1008d1a2e76e70b36f30

Merge "Remove the vp8_vpxyv12_copy_y_neon."

Merge "thumb: Clarify a comment about the conversions that are done"

Cleaning up vp9_mcomp.c.

Change-Id: I24a3f0f215d04fb8888c303642dd5663cc593317

Merge "Remove unused functions"

Merge "Cleaning up vp9_diamond_search_sad_c & vp9_diamond_search_sadx4."

Fix the md5 mismatch for some scale cases.

Fixes issue #731
Change-Id: Id313e84b8fb4ff20f6a4e1ed11cb601927888318

Remove non-420 configure option.

Change-Id: Ic130a77cb77c780bea9d5a3fc7c7dccd3c77f0dd

Merge "Adding a configure flag to control WebM container support"

thumb: Clarify a comment about the conversions that are done

The comment made it look like the condition code was dropped from
the extra add instruction, while it actually was handled properly.

Thus, the comment was misleading while the code itself did the right
thing.

Also clarify the comment indicating that we use the full three-operand
form of the add instruction.

Change-Id: I2c1ac6ac4fedf262d104ea30a6c005febc74de9c

tokenize: quiet -Warray-bounds warnings

eob is limited by GetCoeffs

Change-Id: Ie5c0d024796fe6c9b2db0374892544e421bd5d09

rdopt: quiet -Warray-bounds warnings

eob is limited by GetCoeffs

Change-Id: Id48a92e600375a1d4fb956757c93c91ebb5df59a

onyx_if: quiet -Warray-bounds warnings

'number_of_layers' is range checked before assignment from the user
config.

Change-Id: Idefdaceb8736f126fa7c647da2b047dafb56ea52

svc_encodeframe: quiet -Warray-bounds warnings

si->layers is validated prior to assignment; add an assertion prior to
first use as a loop bound

Change-Id: I727d4e24be3c352c3e5320d92e89d3f787944812

Add bit-stream write flag to nonrd_use_partition

This is for the purpose to test variable partition choices for the
same 64x64 block.

Change-Id: I0fab7351a62e7d3dfccd667e908bccb996ee677e

Remove the vp8_vpxyv12_copy_y_neon.

vp8_vpxyv12_copy_y_neon is slower than vp8_vpxyv12_copy_y_c.

Change-Id: I7ba860619dc0714d700f232242ce915620461587

Calculate rate_mv after sub-pixel search in non-rd mode

Modified the non-rd rate_mv calculation and moved it after sub-pixel
motion search is done.

Change-Id: I6a5cbd3d67ff0a86b3693e25ecf1fd77b16d708c

Remove unused functions

Change-Id: Iec885af58c95e65814fcb6ab528ce4e0874eb573

Merge "y4minput: add some tolerance for read short counts"

Adding a configure flag to control WebM container support

Adding a --(enable|disable)-webm-io flag to control WebM container input and
output support. For now, enabling WebM IO by default only when there is a C++
compiler. Doing so because eventually we will move WebM IO to libwebm and it
is built using C++.

Change-Id: I210ac36c23528e382ed41d3c4322291720481492

Merge "Properly reset the block coding skip flag array in non-RD mode"

Merge "change to save rdmult value correctly"

Properly reset the block coding skip flag array in non-RD mode

The block coding skip flags are assigned in the normal RD mode
decision loop. They are then used in the final encoding stage.
In the non-RD mode decision, the forward transform and quantization
stages are replaced by modeling based on SSE and variance of
prediction residues. This commit applies reset to this array in
the non-RD coding mode.

Change-Id: I66584669b035e9c8ac23e95047849ff277472742

Merge "Use of inline instead of INLINE."

change to save rdmult value correctly

This commit moves the position where rdmult is saved to make sure it
is the correct value. Prior, an uninitialized value may be saved and
restored.

This addresses issue:
https://code.google.com/p/webm/issues/detail?id=733

Change-Id: I436407f289169bc63da3c5a6bf609bed16cb71b5

Merge "Remove deprecated function nonrd_use_fixed_partition"

Merge "Remove unused comment"