summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2016-09-15Enable ssse3 bilinear testsJohann
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
2016-09-15Add vp8_bilinear_filter testJohann
Build out the sixtap_predict test because the filters are interchangeable. Add verbose failures and border checking. Change-Id: I962f50041750dca6f8d0cd35a943424cf82ddcb1
2016-09-16Merge "Revert "Restore vp8_sixtap_predict4x4_neon""James Zern
2016-09-16Revert "Restore vp8_sixtap_predict4x4_neon"Johann Koenig
This reverts commit d9dce2f48eed1368a44c368fa87a506bd89ffec5. Appears to be failing the SixtapPredict tests in some configurations and possibly test vectors as well. Change-Id: Ica6aa83ebac47d0a76e451846e7da67b1c17a7d7
2016-09-16Merge "Restore vp8_bilinear_predict4x4_neon"Johann Koenig
2016-09-16Merge "zero structures completely"Johann Koenig
2016-09-15Restore vp8_bilinear_predict4x4_neonJohann
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
2016-09-16Merge "Restore vp8_sixtap_predict4x4_neon"Johann Koenig
2016-09-16zero structures completelyJohann
Use vp[89]_zero when possible. Expand the {} set when neither is available or nearby. Change-Id: Ifc1f46f60100916cd798bf7be3a10f09321c99bd
2016-09-16vp8 postproc: expand CONFIG_POSTPROC guardJohann
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
2016-09-15altref test: comment out 'pass'Johann
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
2016-09-15Merge "vp9 frame parallel test: Initialize cfg differently"Johann Koenig
2016-09-15vp9: Small code cleanup.Marco
Remove the experiment LIMIT_QP_ONEPASS_VBR_LAG, as its not currently used and no plan to use in near future. Change-Id: Ib069f8d7225195be04b765d0ab477510dfba6a3b
2016-09-15Restore vp8_sixtap_predict4x4_neonJohann
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. The store, when unaligned, has a version that is ~25% slower but safe when xoffset = 0 (second pass filter only). When the first pass filter (or both) are in play, the new version is almost identical in speed. Worst case performance (both filters, unaligned stores) is roughly 3-4x faster than C. BUG=webm:817 BUG=webm:1273 Change-Id: I1e490e94453e0872151fe0dafb05557463f6247d
2016-09-15vp9 frame parallel test: Initialize cfg differentlyJohann
Use the canonical 'vpx_codec_dec_cfg_t()' as opposed to 'vp9_zero()' which just hammered everything to 0. Change-Id: Id820efef700ad92a625797f8fd58e465b15eeca4
2016-09-15Merge "Documentation for building unit tests for Android"Johann Koenig
2016-09-15Documentation for building unit tests for AndroidJohann
BUG=webm:1258 Change-Id: Iea142f7b0df0e047720e8c5362464932de57d564
2016-09-14Merge "cosmetics,vp8: join some lines, fix table format"James Zern
2016-09-13vp8 decoder: cast decoding_thread_count to intJohann
For some reason allocated_decoding_thread_count is signed, but decoding_thread_count is not. Cleans -Wextra/-Wsign-compare: comparison between signed and unsigned integer expressions Change-Id: Id0ada78100acff27c1c4ed7493c563d13c55cdcd
2016-09-13vp9 frame parallel test: Initialize cfg to 0Johann
Use vp9_zero() to set every element. Cleans -Wextra/-Wmissing-field-initializers: missing initializer for member ‘vpx_codec_dec_cfg::w’ missing initializer for member ‘vpx_codec_dec_cfg::h’ Change-Id: I5b41ce7d55a912e29b1d4c3e840cea80e8510fbe
2016-09-13vp9cx_set_ref.c: remove unused 'cfg' parameterJohann
Cleans -Wextra/-Wunused-parameter warning: warning: unused parameter ‘cfg’ Change-Id: I84eae57a50306cb66c625bb648b0a330678818db
2016-09-13webmenc: remove unused 'fps' parameterJohann
Cleans -Wextra/-Wunused-parameter warning: warning: unused parameter ‘fps’ Change-Id: Ia5f9338f11ae8d0708a87c6d4e7d7e924fc3b19b
2016-09-09cosmetics,vp8: join some lines, fix table formatJames Zern
Change-Id: Idcf3b68f0e59bd74c9d332bbd4a7c1484ddb691a
2016-09-09vp8: Set the skin model to mode 1.Marco
This change was reverted before due to a hangouts encode-time regression investigation. But since then this change has been cleared of causing any noticeable regression. This mode reduces some false detection, and uses the same model as in vp9. Change-Id: I9c82a748c5f601d0aca9f61ee218abfbd58c62bd
2016-09-09Merge "vp8: Remove TSAN warning around end of encode."James Zern
2016-09-08vp8: Remove TSAN warning around end of encode.Alexander Potapenko
Tsan warns when run in one pass and there is a recode loop. Change-Id: Ice2ecb2270f09ebd49efbd49c0e4f77d32e23c0f
2016-09-08Merge "vpx_dsp: added vpx_highbd_idct32x32_1_add_sse2()"James Zern
2016-09-07vpx_mem.c: remove unnecessary inlineJames Zern
these aren't overly speed critical, best to leave it to the compiler. Change-Id: I231c14abee5b845d7b8e8454832f2feb22c6ce45
2016-09-07vpx_dsp: added vpx_highbd_idct32x32_1_add_sse2()Scott LaVarnway
Change-Id: I140d93aebadb0eaf6220881e61a0451450081227
2016-09-03Fix missing write to opsnr in internal statsSarah Parker
Change-Id: I21c8ad0b5ed7f8d843cae45c18f5727bceb8f859
2016-09-03Merge "invalid_file_test: quiet -Wunused-const-variable warnings"James Zern
2016-09-01invalid_file_test: quiet -Wunused-const-variable warningsJames Zern
present when --disable-vp8(-decoder) or --disable-vp9(-decoder) was used Change-Id: I31ebb7a55c6f1af3c744982f56b78e80116cc845
2016-09-01vp8_cx_iface: quiet -Wshorten-64-to-32 warningJames Zern
set_reference_and_update(): use the correct type for flags, vpx_enc_frame_flags_t Change-Id: I257da784537ff18686f6db8665f99af6ea6a86ba
2016-09-01get_cpu_count: quiet -Wshorten-64-to-32 warningsJames Zern
sysconf returns a long; cast (unsigned) dwNumberOfProcessors to int for good measure Change-Id: I1f181d7bd9a060c0898db41f66a5065394afdc4e
2016-09-02Merge changes from topic 'Wundef'Johann Koenig
* changes: Enable -Wundef by default Define VP8_TEMPORAL_ALT_REF to !CONFIG_REALTIME_ONLY Remove CONFIG_DEBUG guards from assert() Remove unused function vpx_de_mblock Fix -Wundef warning for OUTPUT_FPF Fix -Wundef warning for __SANITIZE_ADDRESS__
2016-09-01Merge "Fix formatting in internal stats for vp8 and vp9"Yaowu Xu
2016-09-01Merge "Casts to remove some warnings."Yaowu Xu
2016-09-01Merge "Refactor uv tx size with lookup arrays"Debargha Mukherjee
2016-09-01Merge "Modified resize loop constraints."Paul Wilkins
2016-09-01Casts to remove some warnings.paulwilkins
Added casts to remove warnings: BUG=webm:1274 In regards to the safety of these casts they are of two types:- - Normalized bits per (16x16) MB stored in a 32 bit int (This is safe as bits per MB even with << 9 normalization cant overflow 32 bits. Even raw 12 bits hdr source even would only be 29 bits :- (4+4+12+9) and the encoder imposes much stricter limits than this on max bit rate. - Cast as part of variance calculations. There is an internal cast up to 64 bit for the Sum X Sum calculation, but after normalization dividing by the number of points the result will always be <= the SSE value. Change-Id: I4e700236ed83d6b2b1955e92e84c3b1978b9eaa0
2016-08-31Enable -Wundef by defaultJohann
BUG=webm:1069 Change-Id: I43728f9fd007542718a55d5fdcbc63a8d2f86682
2016-08-31Define VP8_TEMPORAL_ALT_REF to !CONFIG_REALTIME_ONLYJohann
Previously VP8_TEMPORAL_ALT_REF was only defined for non-realtime-only builds. However, its value was checked with #if, not #ifdef. Fixes -Wundef warnings. BUG=webm:1069 Change-Id: If78d8731298f3f0d3662ffa25f973e7adaf67152
2016-08-31Remove CONFIG_DEBUG guards from assert()Johann
When 'NDEBUG' is set, assert() generates no code. Change-Id: Icf61cfc1a8f6e5f0770b3626d8c73ae968df1108
2016-08-31Remove unused function vpx_de_mblockJohann
vpx_config.h was not included so CONFIG_POSTPROC was never defined. Change-Id: I777de499823afa286734549a8e7f4a93e7ad97f3
2016-08-31Fix -Wundef warning for OUTPUT_FPFJohann
BUG=webm:1069 Change-Id: I3d13d07cf0934e6e262c8033bd77d7197d03ce21
2016-08-31Fix -Wundef warning for __SANITIZE_ADDRESS__Johann
BUG=webm:1069 Change-Id: Iad8811939a910a8f31cf5788220712a255ddf36a
2016-08-31Merge "Rename test/lpf_8_test.cc to test/lpf_test.cc"Linfeng Zhang
2016-08-31Rename test/lpf_8_test.cc to test/lpf_test.ccLinfeng Zhang
It actually tests all sizes lpf functions. Change-Id: Ie31798f90165e6e0c13cbac0e0ab9648ab568bce
2016-08-31Update NEON transpose functions.Linfeng Zhang
Unify coding style. Change-Id: I5826f40c02c882df7353391e0c9dd6cef6bd4b97
2016-08-31Refactor uv tx size with lookup arraysDebargha Mukherjee
Change-Id: Ife6a3d301c5faaba89d16d188d638631083511f7