summaryrefslogtreecommitdiff
path: root/vp8
AgeCommit message (Collapse)Author
2016-08-22Revert "vp8: Move loopfilter synchronization to end of encode_frame call."Marco Paniconi
This reverts commit c2fe9acceda922ca1d9f0d6185b340560b93597a. This change break linux browser test in chromium: https://build.chromium.org/p/chromium.webrtc/builders/Linux%20Tester Change-Id: I226782fad480c17a99ec6c785ad93cf4ab88f0ae
2016-08-16vp8: Move loopfilter synchronization to end of encode_frame call.Marco
Change-Id: I5bdfea7f51df1f1fa5d9c1597e96988acce6c2f2
2016-08-10Align thread entry point stackAleksey Vasenev
_beginthreadex does not align the stack on 16-byte boundary as expected by gcc. On x86 targets, the force_align_arg_pointer attribute may be applied to individual function definitions, generating an alternate prologue and epilogue that realigns the run-time stack if necessary. This supports mixing legacy codes that run with a 4-byte aligned stack with modern codes that keep a 16-byte stack for SSE compatibility. https://gcc.gnu.org/onlinedocs/gcc/x86-Function-Attributes.html Change-Id: Ie4e4ab32948c238fa87054d5664189972ca6708e Signed-off-by: Aleksey Vasenev <margtu-fivt@ya.ru>
2016-08-04Remove armv6 targetJohann
Change-Id: I1fa81cc9cabf362a185fc3a53f1e58de533a41e5
2016-08-03Pad 'Left' when building under ASanJohann
The neon intrinsics are not able to load just the 4 values that are used. In vpx_dsp/arm/intrapred_neon.c:dc_4x4 it loads 8 values for both the 'above' and 'left' computations, but only uses the sum of the first 4 values. BUG=webm:1268 Change-Id: I937113d7e3a21e25bebde3593de0446bf6b0115a
2016-07-28vp8: Switch skin model to mode 0 to save some cycle.JackyChen
This change will speed up vp8 encoder by 1.5% ~ 2% on linux. No much speed change on Mac. Change-Id: Id957f19ddd89805baa2af84c5027d52d9a48553f
2016-07-23vp8/decodeframe: fix signed/unsigned comparisonJames Zern
quiets a visual studio warning Change-Id: Ic7725616bc2cb837e6f79294d4fcff36b67af834
2016-07-23vp8/postproc.c: disable clang-format for RGB_TO_YUVclang-format
Change-Id: Id2a936301ec1e3d5648b4f8adbf4e6625002589d
2016-07-23Merge changes I0089e884,Icb0ecb9eJames Zern
* changes: vp8/postproc: fix implicit float conversion blockiness_test: fix implicit float conversion
2016-07-23Merge "vp8:fix threading issues"James Bankoski
2016-07-23vp8:fix threading issuesJim Bankoski
1 - stops de allocating before threads are closed. 2 - limits threads to mb_rows when mb_rows < partitions BUG=webm:851 Change-Id: I7ead53e80cc0f8c2e4c1c53506eff8431de2a37e
2016-07-22vp8/postproc: fix implicit float conversionJames Zern
float->int as reported by -Wfloat-conversion Change-Id: I0089e8847b218c47526bcfbb0fffd9aad7c5adb3
2016-07-21Add back header in threading.hYunqing Wang
Added back the header needed in threading.h Change-Id: I2ce66ad4fe58004997623f6c3f3b8dd11640aa98
2016-07-21Revert "Amend and improve VP8 multithreading implementation"Yunqing Wang
Reverted the patch because of possible performance issue. Change-Id: I49944f827ccd38ed194c9f8d9cb9036fa9bf79e1
2016-07-20vp8/onyx_if.c: rework #if's to avoid dangling else'sJames Zern
Change-Id: Ieda8958a3da1000424fcff91a1315d0049612202
2016-07-20vp8/bitstream.c: rework #if to avoid dangling elseJames Zern
Change-Id: I9178ae75876f3df3fa3271314db39830552b9549
2016-07-19vp8/{bitstream,rdopt},y4minput: correct break placementJames Zern
these should be placed within {}s when present Change-Id: If00e9766fa8cb039cc070467f353a468f99460fb
2016-07-19vp8: simplify a few #if'sJames Zern
bitstream.c: asserts are disabled when CONFIG_DEBUG is unset vp8_dx_iface.c: split |s into 2 statements across #if bounds Change-Id: I307d1e969134db5c9c0edd7690589b6b29116cbd
2016-07-19vp8: remove extra semicolonsJames Zern
Change-Id: I84e1a293ee033865f82c244e8aaaadfb2fb27e63
2016-07-19vp8: apply clang-tidy google-readability-braces-around-statementsclang-tidy
applied against an x86_64 configure clang-tidy-3.7.1 \ -checks='-*,google-readability-braces-around-statements' \ -header-filter='.*' -fix + clang-format afterward Change-Id: I6694edeaee89b58b8b3082187e6756561136b459
2016-07-18prepend ++ instead of post in for loops.Jim Bankoski
Applied the following regex : search for: (for.*\(.*;.*;) ([a-zA-Z_]*)\+\+\) replace with: \1 ++\2) This misses some for loops: ie : for (mb_col = 0; mb_col < oci->mb_cols; mb_col++, mi++) Change-Id: Icf5f6fb93cced0992e0bb71d2241780f7fb1f0a8
2016-07-15vp8: apply clang-formatclang-format
Change-Id: I7605b6678014a5426ceb45c27b54885e0c4e06ed
2016-07-15addnoise : clear out static size for generated noiseJim Bankoski
Change-Id: I5d4343f2da9cd4b01dd37be7a048d159fec109d1
2016-07-15postproc : fix function parameters for noise functions.Jim Bankoski
Change-Id: I582b6307f28bfc987dcf8910379a52c6f679173c
2016-07-13postproc: noise style fixes.Jim Bankoski
Change-Id: Ifdcb36b8e77b65faeeb10644256e175acb32275d
2016-07-13postproc - move filling of noise buffer to vpx_dsp.Jim Bankoski
Change-Id: I63ba35dc0ae9286c9812367a531e01d79a4c1635
2016-07-12deblock filter : moved from vp8 code branchJim Bankoski
The deblocking filters used in vp8 have been moved to vpx_dsp for use by both vp8 and vp9. Change-Id: I5209d76edafc894b550f751fc76d3aa6799b392d
2016-07-01remove *debugmodes.c from the default buildJames Zern
these are debug-only modules that can be added in manually when needed. leave a reference in vp8_common.mk / vp9_common.mk for easy addition. quiets -Wmissing-prototypes warning BUG=b/29584271 Change-Id: Ifc8637d877edfbd562b34dc5c540428bba7951fc
2016-07-01vp8/common/reconintra4x4.c: add missing includeJames Zern
quiets -Wmissing-prototypes warning BUG=b/29584271 Change-Id: I806e3475ebee579dce0073dd1784a7c2899e7de0
2016-06-29Merge "Remove effectless initialization"Yaowu Xu
2016-06-28Remove effectless initializationYaowu Xu
Change-Id: Iec117841a7ecf6f99d2b718057d8646e221c5c64
2016-06-27*.asm: normalize label formatJames Zern
add a trailing ':', though it's optional with the tools we support, it's more common to use it to mark a label. this also quiets the orphan-labels warning with nasm/yasm. BUG=b/29583530 Change-Id: I46e95255e12026dd542d9838e2dd3fbddf7b56e2
2016-06-24Port metric computation changes from nextgenv2Yaowu Xu
Change-Id: I4aceffcdf7af59ffeb51984f0345c3a4c7e76a9f
2016-06-23vp8 machine setup: mark unused variableJohann
When building without multithreading and for a non-arm, non-x86 system, ctx is unused. Cleans up -Wextra warning: unused parameter ‘ctx’ [-Werror=unused-parameter] Change-Id: Ifddff89d2ebd45f7d71e3d415a8f2415dd818957
2016-06-23vp8 realtime encoder: mark unused variableJohann
'duration' is not used in realtime-only mode: Cleans up -Wextra warning: unused parameter 'duration' [-Wunused-parameter] Change-Id: I827dfe59ebcdc72c5a93fdf7e5aca063433914b1
2016-06-22vp8 error concealment: remove unused variablesJohann
vp8_conceal_corrupt_mb is an empty function. Remove it entirely. Cleans up -Wextra warnings: unused parameter 'mi_stride' [-Wunused-parameter] unused parameter 'xd' [-Wunused-parameter] Change-Id: Idca7ef4508fae2b4b76a40d44507522a72ccc2c8
2016-06-17Output frames in first pass for VPX_DL_REALTIME.Tom Finegan
Since combining VPX_DL_REALTIME with VPX_RC_FIRST_PASS is basically nonsense, ignore the user's pass setting when this happens and behave as if the requested encode is a single pass encode. BUG=webm:1233 Change-Id: I5ee4c4e5838c4ca6d24988890aae490b10826db2
2016-06-17Merge "vp8: correct function return types"Johann Koenig
2016-06-17Merge "VP8_COMP: make frames_since_golden signed"Johann Koenig
2016-06-16Merge "vp8_[cd]x_iface.c: Initialize structures to 0"Johann Koenig
2016-06-16Merge "vp8 stats file: initialize to 0"Johann Koenig
2016-06-16vp8_[cd]x_iface.c: Initialize structures to 0Johann
Use vp8_zero() or specify every element. Cleans warning in Android build: missing field 'deblocking_level' initializer vp8_ppflags_t flags = {0}; ^ missing field 'sz' initializer {0}, /* rc_twopass_stats_in */ ^ missing field 'sz' initializer {0}, /* rc_firstpass_mb_stats_in */ ^ missing field 'layer_target_bitrate' initializer }}, ^ missing field 'deblocking_level' initializer vp8_ppflags_t flags = {0}; ^ missing field 'mr_get_mem_loc' initializer } ^ Change-Id: Iaedde9a77faac7a40316aee67d60d02ba7313500
2016-06-16VP8_COMP: make frames_since_golden signedJohann
This value is signed in vp9/10 Cleans warning in Android build: comparison of integers of different signs: 'unsigned int' and 'int' if (cpi->frames_since_golden == (cpi->current_gf_interval >> 1)) ~~~~~~~~~~~~~~~~~~~~~~~~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Change-Id: Ie137724982f3a46c8c1820548c1960d62a4e96f2
2016-06-16vp8: correct function return typesJohann
left_above_mv and above_block_mv return as_int as_int is defined as uint32_t in vp8/common/mv.h Cleans up -Wextra warnings: signed and unsigned type in conditional expression this_mv->as_int = col ? d[-1].bmi.mv.as_int : left_block_mv(mic, i); ^ this_mv->as_int = row ? d[-4].bmi.mv.as_int : above_block_mv(mic, i, mis); ^ left_mv.as_int = col ? d[-1].bmi.mv.as_int : ^ Change-Id: Ia043764e4ce93d2152d2269b1c7b28b5d5f814cf
2016-06-16Merge "vp8_change_config: fix unsigned/signed comparison"Johann Koenig
2016-06-16vp8 stats file: initialize to 0Johann
Cleans up -Wextra warning: missing initializer for field ‘intra_error’ of ‘FIRSTPASS_STATS’ Change-Id: I42c1413234aba207f8e89f2e040e14a81bca511e
2016-06-16vp8_change_config: fix unsigned/signed comparisonJohann
Use ~15 instead of 0x..F0 Cleans warning in Android build: comparison of integers of different signs: 'unsigned int' and 'int' if (((cm->Width + 15) & 0xfffffff0) != ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^ comparison of integers of different signs: 'unsigned int' and 'int' ((cm->Height + 15) & 0xfffffff0) != ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^ Change-Id: Iac25839cde3425b7b9db7f33740dc46a551b7546
2016-06-16Fix ubsan warning: vp8/encoder/denoising.cYaowu Xu
BUG=webm:1219 Change-Id: Iebdedfb27c70909c0849e169473a2a3ec9f58cc7
2016-06-16Fix ubsan warning: vp8/encoder/onyx_ifcYaowu Xu
BUG=https://bugs.chromium.org/p/webm/issues/detail?id=1219 Change-Id: I4362a1cf79fdcb478b29ddbcdf8091c5f5d10645
2016-06-15Merge "Match prev_ to number_of_layers"Johann Koenig
generated by cgit v1.2.3 (git 2.25.1) at 2024-10-13 01:20:28 +0000