This will speed up lv_map by 4.3%

Change-Id: I886f67b9f1ae2e5f567e114fa34cf3ba437b8381

Change-Id: I6e9e1c94f688ed84c49d3b861dbac15c37248827

For sub-8x8 blocks, scan loop should have at least one iteration.
Previously, loop would be skipped if col_offset == 1 and n8_w == 1.

This change appears to have neutral effect on bdrate (+0.01).  This could
mean that extra MV candidates are not helping that much for sub-8x8.

Change-Id: Ic778312b39113ab8f348f5c7c18813593a831b7c

The orders_32x128 and orders_128x32 variables are only used when
CONFIG_EXT_PARTITION is false. Only define them in that case to avoid
a compiler warning.

Change-Id: Iebe297eff36b139caa83d039101cf16eb88ca8f8

The change is purely an encoder-side tuning, which constrains a GF
group interval to be either 8, or 12, or a number larger than 12
(maximum of 16). The change is under ext_refs.

This work has been part of the summer internship effort by chendixi@.

Further, this CL removes the experiment of flex_refs and merges it
into ext_refs, as flex_refs has been completely focused on the encoder
side optimization for ext_refs.

On Google test sets, above (stillness + interval enforcement) achieve
a coding gain in BDRate as follows, compared against the AV1 baseline
with all tools enabled by default (excluding ext-tx, global-motion,
and convolve-round for speed concern):

lowres: avg_psnr -0.193%; ovr_psnr -0.370%; ssim -0.332%
midres: avg_psnr -0.556%; ovr_psnr -0.680%; ssim -0.798%

Change-Id: Ia992d8ef0d82cc0ef2caba5eec9b26f0358e6bde

This will speed up lv_map exp by 8%

Change-Id: Ideaebfd855a3b2e7c2a07ac028407e7336b86697

This will speed up lv_map encoder time by 1.5%

Change-Id: Ia9f7634bae221d2fb8fb7dc6f3202c6bacdd5b32

the coverage flags are required at link time as well as compile time

Change-Id: Ic581f8ce747615f1fa1dc13d989d6e233e982572

CONFIG_UV_LVL --> CONFIG_LOOPFILTER_LEVEL
Change name because the experiment combines filter level search
in U, V plane, and horizontal, vertical search in Y plane.

Change-Id: Ia05b8f80cd93b60a0305eb94e5783559ef904c13

A new experiment for deblocking filter that separates vertical
and horizontal filter strengths. This experiment is based on the
assumption that non-flatness characteristics of vertical and
horizontal direction may differ. Therefore selecting different
filter strengths for vertical and horizontal can improve deblocking
performance.

The process of finding proper filter strength:
1. Search through the filter level under the constraint that
   (vertical == horizontal), and find the best solution.
2. Fix vertical level as the best solution found in step 1 and vary
   horizontal level to find the best value.
3. Fix the selected horizontal level, vary vertical level to find
   its best value.

The experiment is working with UV_LVL, sharing the same config flag.
The searching for horizontal and vertical filter strength only applies
on Y plane for now.

The experimental flag should be changed to filter_level later.

Change-Id: I164eec8d3ccb3da7ff109c5c55f4b52c1536ddf1

Currently nothing forbids wedge from being signalled when
the block is > 32X32, even though there is no corresponding wedge
mask for that block size.

BUG=aomedia:640
BUG=aomedia:636

Change-Id: I538be0229a12b5ef01b2e5a950c9f16ef9a5c51e

BUG=aomedia:697

Change-Id: Id7e971e515c9faa3df32e162befd57907278549d

This commit resolves an encoding failure issue due to the
tokenization in palette when the lv-map experiment is turned on.

BUG=aomedia:689

Change-Id: I8e476ae81e4d5d11c6aab7611dff9deb7fa63e9e

Mrc_tx is currently not working with lv_map, supertx and
coef_interleave and requires var_tx.

Change-Id: I6e9d7a54ed10a7a4e5c2088d5907ac8dc610a474

BUG=aomedia:685

Change-Id: I8490af713c53e8c420ef9dffa7c27bb71bbfca54

Coding gain becomes tiny on top of other experiments.

Change-Id: Ia89b1c2a2653f3833dff8ac8bb612eaa3ba18446

The AVX2 code only supports DCT_DCT. For other transform types, use
the C fallback.

Change-Id: I6b472ebd7d963c02aae80ff5846b7f2dcaf092ea

In patch 1cfe474b ("Simplify motion var parameters in reconinter.h"),
I moved some code from the header down into
reconinter.c. Unfortunately, I managed to drop it inside an #ifdef
CONFIG_EXT_INTER/#endif pair, which means the build won't work if
compiled with --disable-ext_inter.

This patch moves the hunk of code to above the #ifdef.

BUG=aomedia:694

Change-Id: I1e83bb500bcb607ee5911027b7c4787184703a57

Avoid changing the public API based on an experiment flag:

- Remove aom_config.h include from aom_image.h.
- Unconditionally include the existing API additions.
- Add some missing doc comments (silences Doxygen warnings).
- Return an error when using ctrl flags that require
  CONFIG_COLORSPACE_HEADERS when it's not enabled.
- Move colorspace_headers to the correct section of configure.
- Move CONFIG_COLORSPACE_HEADERS to the correct section of
  aom_config_defaults.cmake.
- clang-format style check appeasement

Change-Id: I7b5d72c6f9f1a5561409d7813ba59180d98d8805

Change-Id: Ib40f6be7d86704d5a5dd4f5bc7dd7331f18894df

Change-Id: Ib9dd9aaa5240ac7c90f9c276ae8ef9055fcf1fdd

Compute base level's count/mag together.
This will speed up lv_map exp by 5%

Change-Id: I479098ed2637fc05d454fb0a7dc9f09aa20401f3

This speed up lv_map exp by 5%

Change-Id: Ic71e55c9e777031fe671589a96d8f2053f4d65a5

Change-Id: Ibe9908fac4920686eaee6e7b261e0ce29241cf4c

Use --tune=[cdef-dist|daala-dist] to enable them.

Also, this commit set the use_activity_masking of PVQ as 0 by deafult,
which means that PVQ assumes daala-dist is not used by default.

Since we're currently not signaling which metric the encoder did use
in the bitstream, the compile flag AV1_PVQ_ENABLE_ACTIVITY_MASKING will tell PVQ
whether daala-dist is used or not.

This commit is the last part of prep-work to remove DIST_8X8, CDEF_DIST,
and DAALA_DIST experimental flags.

Change-Id: Ia465b4d6fe64aac7f04852c8f9f4bac3409d2435

Group and sort the CMake configuration changes in
aom_configure.cmake, and make sure everything that
changes a CONFIG_ variable is taken care of before
inclusion of cpu.cmake (which controls the RTCD
versions of the CONFIG_ vars).

Change-Id: Ib8069721de63c332392b7216fe3f194a077eb73c

Also, move body of update_cfl_costs() to av1_fill_mode_rates().

Results on Subset1 (Compared to 1cfe474b with CFL enabled)

  PSNR | PSNR Cb | PSNR Cr | PSNR HVS |   SSIM | MS SSIM | CIEDE 2000
0.0000 |  0.0000 |  0.0000 |   0.0000 | 0.0000 |  0.0000 |     0.0000

No change in bitstream, for an average encode speed-up of 2.3%.

Change-Id: I3948abcd70cfecad8086edfe4c45552b576ae06f

Currently the rate is never computed for the palette color indices.
The code to compute the rate is inside av1_tokenize_palette_sb
when dry_run == DRY_RUN_COSTCOEFFS, but av1_tokenize_palette_sb is
only called when !dry_run.

Change-Id: Ie33eae9e4bcf1997a22dc939f31001334cb2c399

Even if 'dist-8x8' is enabled with configure,
the dist-8x8 is not acutally enabled (so, no change in encoding behaviour)
until the command line option, '--enable-dist-8x8=1" is used.

The cdef-dist and daala-dist can not be enabled by a command line option yet.

This commit is a part of prep-work to remove DIST_8X8, CDEF_DIST,
and DAALA_DIST experimental flags.

Change-Id: I5c2df90f837b32f44e756572a19272dfb4c3dff4

Added 'cdef-dist' and 'daala-dist' options to --tune-metric'.
(Actual bodies of the options will be added in later separate commits)

Also, if --tune=cdef-dist or --tune=daala-dist,
enable-dist-8x8 is automatically turned on.

This commit is a part of prep-work to remove DIST_8X8, CDEF_DIST,
and DAALA_DIST experimental flags.

Change-Id: I0790b9537dfaf166cb59741997ae27da10697a33

Change-Id: I632a1f42eff2e41206d6023505c3c26dfc4698c2

Change-Id: Id191159a851513b6eb3f2ddd104a1299eed59292

Change-Id: Iaa464fdd36c9a97070517757645da68ef009b82c

Change-Id: Idfa722cf2d1ed2f73630c7480ce166286955e1c5

Change-Id: I58cdf4d1260759f085e12e871d48f59a89a346c4

The tables for guided filter in loop-restoration are changed
to use a max of 5x5 windows (or radius 2).
The aim is to reduce the gate count for hardware implementation.

Change-Id: I7178d6ac09e4731a626f9bccf5151467c63e00c3

Fix a memory corruption introduced by commit
f190a165

On some cases, when empty tiles are present,
av1_set_across_tile_boundary_info() writes beyond mode info array.

Sanity check added.

BUG=aomedia:690

Change-Id: I8d87a2e183c69d0ed039c8b2f77b30434f837974

When ext-comp-refs is enabled, one-sided-compound is enabled by default,
which ensures the use of ext-comp-refs is an extension of
one-sided-compound. Both coding tools allow the use of same-sided
reference frame pairs for compound prediction.

Also, remove the dependency of ext-comp-refs on var-refs, i.e. these two
coding tools can be independently enabled. They can still work together
if both are enabled simultaneously.

Change-Id: I3134e7e2956dc35d557fe814f5d801d473683650

Before this patch, the functions build_inter_predictors, allow_warp
and av1_make_inter_predictor took two parameters (if
CONFIG_MOTION_VAR): mi_col_offset and mi_row_offset. These were used
for two things: they were used to look up a MODE_INFO structure and
the code figured out it was doing OBMC if at least one offset was
nonzero.

This patch slightly simplifies things by passing the relevant mi and a
flag saying whether we're doing OBMC instead. This means the
allow_warp function actually loses a parameter.

It also moves all mention of the functions out of the header and into
reconinter.c since they are only used in that compilation unit.

Change-Id: I8dedad9d7384ee0aa17b977e92ea7bfa83a532ef

adjustments made by aom_configure.cmake need to be done prior to
including cpu.cmake

Change-Id: I482905414da4b1e112066d127cd71f8eae9dd504