Merged
Conversation
Remove the experiment LIMIT_QP_ONEPASS_VBR_LAG, as its not currently used and no plan to use in near future. Change-Id: Ib069f8d7225195be04b765d0ab477510dfba6a3b
All the other test which do not use 'pass' (which appears to be almost all of them) do this. Cleans -Wextra/-Wunused-parameter: unused parameter ‘pass’ Change-Id: I1ff3acf3f3d1e831f94dcb00ea36337afe0aefe0
postproc.c is overloaded and used for both postproc and internal stats. If only --enable-internal-stats is specified there are issues with non-existent struct members and unused functions. Change-Id: I82367f1ffce659c3918c9f964dbce94a716fbb89
Use vp[89]_zero when possible.
Expand the {} set when neither is available or nearby.
Change-Id: Ifc1f46f60100916cd798bf7be3a10f09321c99bd
This function was removed when clang started introducing alignment hints which caused the 32 bit vld1_lane_u32/vst1_lane_u32 to fail: https://llvm.org/bugs/show_bug.cgi?id=24421 The load has been rendered safe with an implementation ~indiscernible performance-wise that uses _u8 and over-reads just a touch. It is still ~5x faster than C in the unaligned case and doing both filters. BUG=webm:892 BUG=webm:1273 Change-Id: Icf7167189391b46202f47233bb585c24c42bcc36
This reverts commit d9dce2f. Appears to be failing the SixtapPredict tests in some configurations and possibly test vectors as well. Change-Id: Ica6aa83ebac47d0a76e451846e7da67b1c17a7d7
Build out the sixtap_predict test because the filters are interchangeable. Add verbose failures and border checking. Change-Id: I962f50041750dca6f8d0cd35a943424cf82ddcb1
The code only has issues when xoffset == 0 and yoffset == 0 which represents a simple copy. Presumably this case does not need to be handled because the issue has existed since 2010. BUG=webm:1287 Change-Id: Ic47e2653f3b729e99b40e53d8d2d8d1501edaaa9
* changes: apply clang-format .clang-format: update to 3.8.1
assume __clang_major__==0 has the latest version of _mm256_broadcastsi128_si256. fixes builds with custom clang toolchains. BUG=b/30970831 Change-Id: I90becd56278e4716bd46e2ba9d910af977e8dfa6
Extract shared code so later lpf size 4 and 8 functions can reuse. Change-Id: Ibb43ef1fd8651bd2e32fcc4c56cf6fa7ca237401
quiets a -Wunused-function warning for arm targets Change-Id: I293a7e3d3d7d61d6af2fbedad5e8c25126c418b6
Change-Id: I495aa9cec2b2b8f1ae69bdab8b3feeca76358472
The vp9_mv_class0_tree is a balanced tree with two leafs and can simply be coded as a boolean with probability class0[0]. Change-Id: If294dac825a5f945371092c74aa8e3f84cd962b6 (cherry picked from commit be8a8ab62ebdd111c6f2e9a33b15630570671eba)
The referenced bug was fixed by saving neon registers. That this had any effect was coincidental. Both chromium and Android build with clang and neither uses this flag. Change-Id: I470247d6fd9226fc207b42a187105581a94badc3
add missing int64 -> uint32 cast; quiets -Wshorten-64-to-32 warnings Change-Id: I4850b36e18dc8b399108342be4bfe0b684aefb78
Also check in 8x8 8-bit transpose NEON intrinsics optimization transpose_u8_8x8() Change-Id: I32d321cf97ea21eab158ac4896990fc9a51681c4
this code is similar to other existing uses and/or vp9 Change-Id: I56e646931379759d9f7332ea6d746060007c75ee
similar to changes that were done in vp9 for encoded frame size reporting. has the side-effect of quieting a -Wshorten-64-to-32 warning. Change-Id: I89f74cb617fc29334ee351dc8dfaa3b8cfd4e5af
all around usage of strtol/strtoul Change-Id: If907c89f107a068987aa71ddd93cee9a7389e4cd
these were moved to vpx_dsp Change-Id: I307b07ae05e2333277d4b7011cba36dcf8409959
Removed a couple of adjustments that no longer move the needle much but complicate the process of tuning. Change-Id: Ie320f5cf155e6aac14a4757ea9ada2cd59f27590
Remove double counting of decay. Limit maximum KF boost. Change-Id: I0fb2344d0f78b5e95bb899dfad12b0ca84034b2c
Switch to using correctly sized inputs and outputs. This simplifies adding tests with varying strides. Change-Id: I716a0d8173dcf6a86d56656ac9d3101b7ec27642
Use *input_block_ for sizeof() calculation like the other test Change-Id: I1e4bd227131662056405af78c5052ad6ef769e9f
The result of the transform is added to the destination buffers. In the existing tests the destination buffer is always empty so that portion of the code was never exercised. Change-Id: I1858c4fed2274f1b9faf834d2ba4186a4510492a
Fixes a case where recode is not triggered based on the value of maxq passed into the recode loop test function. BUG=b/32375284 Change-Id: I15ad985d0525c68e0443cfaf842440d2754b2266
dst += stride behaving better with gcc/clang. Unroll loops. Change-Id: I83f85df2bc9f17c6159542f57680b509395db2b1
Change-Id: Icc3e5aaa6636ffe17dc9da5f7a80afaccbde509a
Allow for passing in the layer bitrates at command line. Fix to allow passing in bitrate for each spatial-temporal layer. Change to some default values for 1 pass cbr mode: spatial scale and qp-max/min. Small fixes to some build warnings. Change-Id: I3f9a776262712480a6570bb863a835b2fc49935a
Permits skipping 0, 1/2 or 3/4 of the frames, corresponding to temporal layers 2, 1 and 0 of a 3-temporal-layer encoding. 1/2 corresponds to TL0 in a 2-layer encoding. Change-Id: I7f6d131f63707e5262fc67d111bfb3a751ede90d
s/inttypes.h/vpx_integer.h/ clear a uint64_t -> double conversion warning Change-Id: I58d108b083787a754152eb79ef6df61c2c5f95b1
A past patch made it so that every frame that had a decode error caused a corrupted frame to be counted. Unfortunately it was possible to get both a decode error and a corrupt frame for the same frame and thus double count an error. This code makes that impossible. Change-Id: Iea973727422a3bf093ffda72fa358a285736048b
Add some checks to the layer bitrates passed in through the command line. Change-Id: I16f270035a6034d63313fe3019aa90dca9a3eefb
Change-Id: I45e1454c3a85e081bfa14386e0248f57e2a91854
Change-Id: I3c90d657cca533264dd62bb7749c53a862d0352a
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
19 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.