GitLab

Add an SSE4.1 lowbd version of the self-guided filter for
loop-restoration, and apply some optimizations to the C
version.

Approximate times per 128x128 / 256x256 tile on the machine
this was developed on:
Previous C:  620us / 2800us
Optimized C: 500us / 2200us ( 24% /  27% faster)
SSE4.1:      147us / 600us  (320% / 370% faster)

Change-Id: I23ff5a5482a191aeb06f9d1f767a9f036bb357fe

Use 255 instead of 256, to restrict to 8-bits.

Only noise level differences in performance.

AWCY:
                   High Latency     Low Latency
  All Keyframes    -0.01            -0.01
  Video overall    -0.01            -0.07

Google Set:
                  All KF            Video
lowres            -0.005            -0.029
midres            -0.008             0.028
hdres             -0.010            -0.022      

Note: By moving from 18-bit to 8-bit and then
cutting off at 255 (this change, overall effect is
noise level too (neutral or better).

Change-Id: I9f2852023015e36c01203bafe486ec400b2ba46f

Change-Id: I9bee09b364223e776cd14ae4a3d47e0c3ffcf008

This fixes the coding failure when both tile-groups and ext-refs
are turned on. It clears the nightly-run unit test failure.

Change-Id: Idb67580ebe3ef625cf3ab18dcf7a274db4dacab3

Change-Id: I49fc7c63b5a8db648856ee046e3209dc577f4067

Delta Q is coded with a single tree context.

Change-Id: Ie1e56c70285e948232049bfb4a51ad0e9ee25f75

Removes redundant deblocking filtering operations and buffer copies,
during search for the restoration filters.
Also, makes the order of cdef and loop-restoration consistent.

BUG=AOMEDIA:373

Change-Id: I571e331f9dfadfd2c1494f279e50510ec212ea29

With this patch, --enable-var-tx only enables recursive transform
partitioning without using rectangular transforms.
To enable use of rectangular transforms in addition, use:
--enable-var-tx --enable-rect-tx

The RD selection process is not fully tested under the var-tx flag
only. We might expect certain performance loss there.

Change-Id: Ie6aa17f1bbc3e8563b9990bc9ff79cc860d9a361

Also handle block centers for sub8x8 blocks correctly.

BDRATE: lowres: -1.308%

Change-Id: I6bdb25bdaceb5086a8b951617fd62ed41cd89023

Change-Id: I4bbe63a4e54dd14a05fca26a11abb5a0d12480d5

BUG=https://bugs.chromium.org/p/aomedia/issues/detail?id=76

Change-Id: Ia857648599f176834307191af6a380b3e411850d

Same sources as CONFIG_AV1 section. Combined them, and fixed
a comment.

Change-Id: I5273849143c0c92a506deeb9241a761e5ee125d3

Change-Id: I0dd17ec7ea59cecf8298831d0e46e971f744c2c7

BUG=https://bugs.chromium.org/p/aomedia/issues/detail?id=76

Change-Id: I3e19eb302eb2f0f5a12243bf093c8f9807d97e91

The use of target_sources in combination with generator expressions
is not supported before v3.5. Set min required version to v3.5.

BUG=https://bugs.chromium.org/p/aomedia/issues/detail?id=76

Change-Id: Ifa1ea25e17ac58d631436c31ab9d4d9231e689f2

Note: CONFIG_FILTER_INTRA is broken in both build systems due
to link errors. This is a parity patch.

BUG=https://bugs.chromium.org/p/aomedia/issues/detail?id=76

Change-Id: I713bc7b6f5f13387bbf6b9e3df22731c22d558cc

- Build and maintain a list of configurable variables.
- Add/remove CONFIG_ variables to sync w/old build system.
- Update defaults for same reason.
- Set RTCD versions of CONFIG_ variables using the CONFIG_
  vars directly.

BUG=https://bugs.chromium.org/p/aomedia/issues/detail?id=76

Change-Id: If3ca6c94a8919d92dcc5c47e0010098f5694b165

BUG=https://bugs.chromium.org/p/aomedia/issues/detail?id=76

Change-Id: Iddb07288166af582b52d75e8b2cb37a55e13a196

BUG=https://bugs.chromium.org/p/aomedia/issues/detail?id=76

Change-Id: I70539530b5db7f971928706122626d524a8fae61

The variables robust and nodesync appear to represent the same concept.
In order to simplify, they are merged into nodesync. This change does
not alter the bitstream as this value is hard coded to 1.

Change-Id: If9b2eb52f628e3f615b6270e26569815d5e79ae0

In Daala, od_pvq_rate depended on the frame type because of H/V pred.
However, H/V pred is not in AV1, as such this code can be removed.

This changes do not impact the bitstream as currently is_keyframe is
hard coded to zero.

Change-Id: Ic4b7c91c004133bc4520870ce39e47cd6a63f298

ext-tile is incompatible, so when it is enabled, tile groups
will be disabled.

Change-Id: Ibf1728a7b576c7c6ad92a2a0de3145a8ecbed6d8

This change improves efficiency and means that tile groups
cause negligible loss (+0.03% BDR) in the current default
encoding configurations (1 tile group no matter how many tiles)
with frame-based wrapping.

In future, when wrapper formats support tile groups, lengths
can be omitted from the last tile in every tile group.

Change-Id: Ia701e6720208fbfa727f9aef0b692a6aafbcc93f

As H/V Prediction is not implemented in AV1, pvq_theta does not need to
consider it when adjusting its parameters.

It is important to understand that the results shown here are not the outcome
of this patch but the outcome of leaving the condition in and enabling
is_keyframe. Currently, this change does not impact the bitstream as
is_keyframe is hard coded to 0 for PVQ.

The following result show that enabled for spatial intra, the H/V pred
tweaks do not yeild benefits.

Enabling the condition for all planes in a keyframe:
Subset1:
pvq@2017-02-28T20:31:50.969Z -> pvq-unreliable-intra@2017-02-28T20:30:32.049Z
PSNR | PSNR Cb | PSNR Cr | PSNR HVS |   SSIM | MS SSIM | CIEDE 2000
0.0896 |  0.0860 | -0.0759 |   0.1608 | 0.1442 |  0.1599 |     0.0147

Enabling the condition for Luma plane in a keyframe:
Subset1:
pvq@2017-02-28T20:31:50.969Z
 -> pvq-unreliable-intra-luma@2017-02-28T20:34:18.029Z
PSNR | PSNR Cb | PSNR Cr | PSNR HVS |   SSIM | MS SSIM | CIEDE 2000
0.0713 |  0.1317 |  0.0717 |   0.1253 | 0.1365 |  0.1406 |     0.0312

Enabling the condition for Chroma planes in a keyframe:
Subset1:
pvq@2017-02-28T20:31:50.969Z -> pvq-unreliable-intra-chroma@2017-02-28T21:09:38.992Z
PSNR | PSNR Cb | PSNR Cr | PSNR HVS |   SSIM | MS SSIM | CIEDE 2000
0.0232 | -0.1240 | -0.1379 |   0.0029 | 0.0187 |  0.0191 |    -0.0916

Based on these results, the condition in the if statement is removed.

Change-Id: Icd2f1313952548939069b4570cf7db04632f35f3

Enable obmc mode only when there are <= 2 left neighbors and <=2
above neighbors. Also disable it when there is no overlappable
neighbors.

Gain in AWCY test: 1.60%, was 1.64% when there is no restriction

Change-Id: I2d82ef4fb4daa9b0843ac8844f99b9f412c4f379

- Quote the generator since it's almost always more than one word.
- Include AOM_TARGET_CPU when autodetected, and omit it in favor
  of including the toolchain file when a toolchain file is
  specified.

BUG=https://bugs.chromium.org/p/aomedia/issues/detail?id=76

Change-Id: If0ba115074b96c93fc93c97747d8f60e919bf6f7

Also adds CONFIG_BITSTREAM_DEBUG support (because it was in
aom_util/aom_util.mk, and that's partly the guidance here).

BUG=https://bugs.chromium.org/p/aomedia/issues/detail?id=76

Change-Id: I971b4025e345fd7d6d766526059f74d8b6ff2ebd

Also, add TODO for rtcd target setup.

BUG=https://bugs.chromium.org/p/aomedia/issues/detail?id=76

Change-Id: Ib2e4ecb30f0d2ce9b538986efdb1ac3117bd59df

- Stop abuse of the aom_util target to expose aom_ports includes
  to IDEs.
- Create aom_ports target only on appropriate platforms.
- Otherwise add the include only sources to the libaom target (aka abuse
  the libaom target instead of the aom_util target).

BUG=https://bugs.chromium.org/p/aomedia/issues/detail?id=76

Change-Id: I82a27a0fc2d6ab13ea273726a4777e21ebdbd247

BUG=https://bugs.chromium.org/p/aomedia/issues/detail?id=76

Change-Id: Ic9072ba53bc7b0b13b227a576ca3d4a691926385

Change-Id: Icc85b0990adfdaf31f246204bee0c9b9d54654a0

Change-Id: I13369c84fa736e0d38f19480068abfb5dc54dcc8

Fixes missing and wrong comments for all of the #endif's in rdopt.c to
improve readability.

Change-Id: Idd557a9e6244bbc335c2a646d150ed59074ee7a1

BUG=https://bugs.chromium.org/p/aomedia/issues/detail?id=76

Change-Id: I8f6dff99e413b799bed67ad0d5ecafc6c0197e22

This patch adds a decoder control that exposes frame data via a simple
 callback.

Change-Id: Icae73ae6b5da8a7783db9fadb1fff4a85d77174b

Fixes the overflow of unsigned int in *state * 1103515245.

Change-Id: Id35b8baa205f1ef8829ae4c8708f0c65edb01ada

1. Adds a limit on number of candidate samples used for the
estimation.
2. Adds a limit on max mv magnitude for use in the least-squares
3. Makes some of the internal variables 32-bit.

Impact on coding efficiency in the noise range.

Change-Id: I8c1c3216368ceb2e3548660a3b8c159df54a8312

Since the tested functions are always forced inline in regular use,
ASM_REGISTER_STATE_CHECK doesn't make sense on this level (the test
should rather be applied to unit tests checking functions making use
of these inlined functions).  The test fails on Win64 because the
Win64 ABI requires xmm6 to xmm15 to be preserved across function
calls, but the ABI is only relevant for non-inlined functions.

BUG=aomedia:371

Change-Id: Icb795083f69465cf09ec8f6871899943efaeaab8

In av1_selfguided_restoration, the values stored into 'dgd' are
unnecessarily rounded twice. This patch replaces this by a single
rounding operation.

Change-Id: I188d283137b74823f5d5447d441250520d6ee294

Always define USE_TXTYPE_SEARCH_FOR_SUB8X8_IN_CB4X4 to avoid the
use of undefined value.

Change-Id: I0ad90c5b5316db231e9538487bb4591dfd6a9ce7