everything outside of third_party should follow 'PointerAlignment:
right' i.e., associate the '*' with the variable

+ add a note about the clang-format that generated this file

Change-Id: I13e3f4f5fb6e22a8fa7fc3d06879c995b7c41a39
(cherry picked from libvpx e429080)

Change-Id: I2fac92e356ed1c78039d63e9f8757e7df3d88756

The SSSE3 version of av1_quantize_fp stored the new value of eob
with a 32bit mov instruction, but the eob values are stored in an
array of uint16_t. This caused the last eob store to write 2 extra
bytes off the end of the eob array.

This doesn't seem to currently be a problem on master, but it
causes errors for my RDO refactoring under certain conditions.

Change-Id: I9f7e9c4e40b37b8a2c232b87d170d24593c1066c

This array was allocated and used to save and restore segmentation map,
however the original segmentation map was never modified between the
calls to save and restore.

Change-Id: Iaf0fbfed733c097e84cf44d2aa6b8f35d2fb456b

Michael Bebenita authored 8 years ago and Michael Bebenita committed 8 years ago

Change-Id: I59bd910f58c218c649a1de2a7b5fae0397e13cb1

The test in arf_freq assumes any no-show frame as ALTREF_FRAME and
then calculate the minimum run between two consecutive ALTREF_FRAME's
based on this assumption. As BWDREF_FRAME is also a no-show frame and
the minimum run between two consecutive BWDREF_FRAME's may vary
between 1 and any arbitrary positive number as long as it does not
exceed the golden frame group interval, this test does not apply to
the experiment of ext-refs. Therefore, ArfFreqTest is disabled when
ext-refs is on.

Change-Id: I970f58fbfb8d2a03160f089748dccf1ad24207ee

Change-Id: I64b236d859a50a265b9b9ee2315bf2549e63881d

Now that all warnings are taken care of, add warning flag -Wshadow to
configure.

Note: Enabling this flag for C++ generates some useless warnings about
some function parameters shadowing class member function names. So, only
enabling this warning for C code.

Change-Id: I53ffb763483fec675a3358c5e64a737a03b93eb1

the bulk was deleted in:
0a9b36b9 Remove unused APIs and skin_detection dead code

Change-Id: I534360ca3d3b379e083b3c79c15eebf0e622fed2

Currently the RD loop traverses 4X8 blocks in inverted N order while
the bitstream stores blocks smaller than 8x8 in Z order. This causes a
discrepancy where the RD loop reads uninitialized data while
performing intra prediction.  As a temporary fix simply disable the
use of the extended right edge for 4X8 blocks, until the bitstream can
be changed to match the logical structure of the blocks.

Change-Id: I44a9e4fc1a15cd551a7b38c3c1227bc5dac77e9a

The decoder is already linear so changing these tables would just create
a mismatch.

Change-Id: Ib888c0dc273e089c38298f569bb35b6e4c32dd60

Change-Id: I6fbe203d37010e47f198215f499c3fa1043fd042

Also port SIMD optimization of weighted sad/variance functions to
av1.
Coding gain improvement: 0.339/0.413/0.328 (lowres/midres/hdres)
Current coding gain: 2.437/2.428/2.294
Encoding time overhead: 17% (soccer_cif), 30% (ped_1080p25), was
12% and 18% without motion search

Change-Id: I101d6ce729f769853756edc8ced6f3a2b8d8f824

Change-Id: I84ff252eb813ce935ad6e17162f4f6fab701dcc7

For a 16x16 pixel block, one needs to allocate 16x16 coefficient
tokens, plus up to 16 eob tokens, per plane. This commit increases
the token allocation size to cover the case where all the transform
blocks are of size 4x4 in format 444.

Change-Id: I5755e6a53771053d51163d01ec1d62e670c5009e

No change in metrics

Change-Id: Ib1dbe41a9e1a564dd9a63a33e2a5315ad6bca70c

No change in metrics

Change-Id: I0da09270d78c3caf78a32a3157f02c87f2232e3e

aom_realloc was allocating 1 byte more than needed every time.
Fixed this, and took this opportunity to do a small refactoring.

Change-Id: I38fcb62b698894acbbab43466c1decd12f906789

Change-Id: I2da9cd5da48ae97e770bccfd1233bcc70b484688

Change-Id: I7c088c55f1c461063976d5bd84ff2026c4f3bc69

Move computing the partition_cdf tables per symbol to
 computing them only when the probabilities are updated.

Change-Id: I442f9230ba00be7f5d0558d7c38d7324ad009ee8

Move computing the inter_ext_tx_cdf tables per symbol to
 computing them only when the probabilities are updated.

Change-Id: I5e1e62f8eae8f6b2edbbd378beeb786649502c10

Move computing the intra_ext_tx_cdf tables per symbol to
 computing them only when the probabilities are updated.

Change-Id: I26d5e419e103093e98a7d896c196176305b50fc9

Change-Id: Iae2366380a8c24e4cde79b6a699b4073b4b7d488

Reduce its size from 270x2 to 90.

Change-Id: Ib6ea59344d7df4cefd2631aa73a1d5e599c212cc

This commit makes the encoder to use different frame context index
for different frame types. In the baseline setting, it sets the
frame context index of the overlay frame to be different from other
regular inter frames. In the ext-refs setting, it further allows
the backward reference frame to use a different index.

It improves the compression performance for both settings.

Baseline
lowres  0.1%
midres  0.15%

ext-refs
lowres  0.42%
midres  0.35%

Change-Id: Ie033fcc16af20dace9a70386dc35dad6c5744f09

SIMD optimization for 1d blending functions in obmc mode, and some
code refactoring and cleanup.

(ped_1080p25.y4m, 150 frame, 2000 tb)
Encoding time overhead: +18.8% -> +18.1%
Decoding time overhead: +21.3% -> +8.7%
Change-Id: I9d856c32136e7e0e6e24ab5520ef901d7b1ee9c8

While we are at it:
- Rename some variables to more meaningful names
- Reuse some common consts from a header instead of redefining them.

Change-Id: I110491c071ae6f68378030493679d49fe670129a

Change-Id: Ibf6985e7b140dc280a74f02084db428a2a85b990

By turning on ext_interp experiment
Encoder time is reduced for 26.7%
Decoder time is reduced for 15.4%

Change-Id: I14e197dd7ed3261339b0643fa4ed0947f74274fc

- Change struct name to all caps SCAN_ORDER to be locally consistent.
- Rename struct pointers to 'scan_order' instead of hard to read short
  names 'so' and 'sc'.

Change-Id: Ia131f14320c1a18aa12654cdb87bf8008878950a

Move from computing the switchable_interp_cdf per symbol to
 computing once per frame when the probabilities are adapted.

Change-Id: I6571126239f0327e22bb09ee8bad94114291683e

Move the av1_indices_from_tree() function from av1/encoder/treewriter.c
 to aom_dsp/prob.c so that it can be used by both the encoder and
 the decoder.

Change-Id: Ie43c599f425c3503b1ff93f0c77b5033a05b1bb4

Add av1_indices_from_tree() function that computes a forward and inverse
 mapping of the tree leaf-node symbols to their in-order traversal.
This is necessary because many of the aom_tree binary trees have their
 leaf nodes out of order (e.g., an in-order traversal of a tree with n
 nodes does not start at symbol 0 and go to symbol n - 1), but the CDFs
 created by tree_to_cdf() are indexed in-order.

Change-Id: Icd0dbed4c171a67c9e84a634106c4fdb5b1b3488

Change-Id: Ic04e180b09745fab2230d05985770c41deea4fad

Without first including ./aom_config.h in aom_dsp/prob.c the memmove
 function is implicitly defined and causes a compiler warning.

Change-Id: I339d0389f10324a1085aba7d6492b2159a14da92

For now this is in its own experiment for benchmarking purposes, but it
will eventually be folded into the main ANS experiment.

Change-Id: Icdb54b6ca3c0191ba717d6d10aaaffb11e2bebea

Change-Id: I96bd5d1fd4d9c3cf9f76fa8d2b0762a7cdc878fd

Change-Id: I371297e6ee000e6dc01ba1544763cbed429b0e5a

Change-Id: I97b4b2cb4c83f2be6a5be65fac6d0017dcf32c67