Re: [libav-devel] [PATCH 1/1] arm: error out if movrelx uses the same dst and GOT register

2016-11-18 Thread Martin Storsjö
On Fri, 18 Nov 2016, Janne Grunau wrote: On 2016-11-18 09:29:36 +0200, Martin Storsjö wrote: On Thu, 17 Nov 2016, Janne Grunau wrote: >On 2016-11-14 00:24:46 +0200, Martin Storsjö wrote: >>Check these constraints even for non-pic-elf configurations, to >>catch the issues without having to buil

Re: [libav-devel] [PATCH 2/2] arm: vp9itxfm: Use cbz for a short jump forward

2016-11-18 Thread Martin Storsjö
On Fri, 18 Nov 2016, Janne Grunau wrote: On 2016-11-18 10:06:39 +0200, Martin Storsjö wrote: --- The uses of cmp rX, #0, beq in the vp9 loop filter can't be converted to cbz, since the branch targets are too far away. --- libavcodec/arm/vp9itxfm_neon.S | 3 +-- 1 file changed, 1 insertion(+),

Re: [libav-devel] [PATCH 1/2] arm: Add cbz/cbnz macros for arm mode

2016-11-18 Thread Martin Storsjö
On Fri, 18 Nov 2016, Janne Grunau wrote: On 2016-11-18 10:06:38 +0200, Martin Storsjö wrote: This allows using the cbz/cbnz instruction in thumb mode, for the cases where the branch target is close enough. --- libavutil/arm/asm.S | 12 1 file changed, 12 insertions(+) diff --git

Re: [libav-devel] [PATCH 1/2] Revert "checkasm: vp9dsp: Benchmark the dc-only version of idct_idct separately"

2016-11-18 Thread Janne Grunau
On 2016-11-18 13:57:04 +0200, Martin Storsjö wrote: > This reverts commit 81d7f0bbca837afda1f7e60d3ae52ab1360ab44b. > > Instead of just benchmarking dc separately, test all relevant subparts. > --- > tests/checkasm/vp9dsp.c | 6 -- > 1 file changed, 6 deletions(-) > > diff --git a/tests/chec

[libav-devel] [PATCH 1/1] arm: error out if movrelx uses the same dst and GOT register

2016-11-18 Thread Janne Grunau
On 2016-11-18 09:29:36 +0200, Martin Storsjö wrote: > On Thu, 17 Nov 2016, Janne Grunau wrote: > > >On 2016-11-14 00:24:46 +0200, Martin Storsjö wrote: > >>Check these constraints even for non-pic-elf configurations, to > >>catch the issues without having to build/test a pic-elf configuration > >>t

Re: [libav-devel] [PATCH 1/2] arm: Add cbz/cbnz macros for arm mode

2016-11-18 Thread Janne Grunau
On 2016-11-18 10:06:38 +0200, Martin Storsjö wrote: > This allows using the cbz/cbnz instruction in thumb mode, for the > cases where the branch target is close enough. > --- > libavutil/arm/asm.S | 12 > 1 file changed, 12 insertions(+) > > diff --git a/libavutil/arm/asm.S b/libavut

Re: [libav-devel] [PATCH 2/2] arm: vp9itxfm: Use cbz for a short jump forward

2016-11-18 Thread Janne Grunau
On 2016-11-18 10:06:39 +0200, Martin Storsjö wrote: > --- > The uses of cmp rX, #0, beq in the vp9 loop filter can't be converted > to cbz, since the branch targets are too far away. > --- > libavcodec/arm/vp9itxfm_neon.S | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/

Re: [libav-devel] [PATCH] arm: vp9itxfm: Simplify the stack alignment code

2016-11-18 Thread Janne Grunau
On 2016-11-18 09:42:15 +0200, Martin Storsjö wrote: > From: Janne Grunau > > This is one instruction less for thumb, and only have got > 1/2 arm/thumb specific instructions. > --- > libavcodec/arm/vp9itxfm_neon.S | 28 > 1 file changed, 12 insertions(+), 16 deletions

Re: [libav-devel] [PATCH] aarch64: vp9itxfm: Use w3 instead of x3 for the int eob parameter

2016-11-18 Thread Janne Grunau
On 2016-11-18 13:54:10 +0200, Martin Storsjö wrote: > The clobbering tests in checkasm are only invoked when testing > correctness, so this bug didn't show up when benchmarking the > dc-only version. > --- > libavcodec/aarch64/vp9itxfm_neon.S | 8 > 1 file changed, 4 insertions(+), 4 dele

Re: [libav-devel] [PATCH 1/3] ape: Restructure DEBUG ifdefs to avoid unused function parameter warnings

2016-11-18 Thread Diego Biurrun
On Fri, Nov 18, 2016 at 10:10:19AM -0500, Vittorio Giovara wrote: > On Fri, Nov 18, 2016 at 4:46 AM, Diego Biurrun wrote: > > --- a/libavformat/ape.c > > +++ b/libavformat/ape.c > > @@ -94,9 +94,9 @@ static int ape_probe(AVProbeData * p) > > > > +#ifdef DEBUG > > static void ape_dumpinfo(AVFormat

Re: [libav-devel] [PATCH 1/3] ape: Restructure DEBUG ifdefs to avoid unused function parameter warnings

2016-11-18 Thread Vittorio Giovara
On Fri, Nov 18, 2016 at 4:46 AM, Diego Biurrun wrote: > --- > libavformat/ape.c | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/libavformat/ape.c b/libavformat/ape.c > index 80e3372..d917815 100644 > --- a/libavformat/ape.c > +++ b/libavformat/ape.c > @@ -94,9 +94,9

Re: [libav-devel] [PATCH] doc: Add libxavs section

2016-11-18 Thread Vittorio Giovara
On Fri, Nov 18, 2016 at 9:45 AM, Diego Biurrun wrote: > --- > doc/general.texi | 8 > 1 file changed, 8 insertions(+) ok -- Vittorio ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH] doc: Add libxavs section

2016-11-18 Thread Diego Biurrun
--- doc/general.texi | 8 1 file changed, 8 insertions(+) diff --git a/doc/general.texi b/doc/general.texi index 7b1f11c..d232300 100644 --- a/doc/general.texi +++ b/doc/general.texi @@ -95,6 +95,14 @@ Go to @url{http://www.wavpack.com/} and follow the instructions for installing the l

[libav-devel] [PATCH 2/2] checkasm: vp9dsp: benchmark all sub-IDCTs (but not WHT or ADST).

2016-11-18 Thread Martin Storsjö
From: "Ronald S. Bultje" --- tests/checkasm/vp9dsp.c | 21 ++--- 1 file changed, 14 insertions(+), 7 deletions(-) diff --git a/tests/checkasm/vp9dsp.c b/tests/checkasm/vp9dsp.c index 690e0cf..25f9dd1 100644 --- a/tests/checkasm/vp9dsp.c +++ b/tests/checkasm/vp9dsp.c @@ -269,14 +

[libav-devel] [PATCH 1/2] Revert "checkasm: vp9dsp: Benchmark the dc-only version of idct_idct separately"

2016-11-18 Thread Martin Storsjö
This reverts commit 81d7f0bbca837afda1f7e60d3ae52ab1360ab44b. Instead of just benchmarking dc separately, test all relevant subparts. --- tests/checkasm/vp9dsp.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/tests/checkasm/vp9dsp.c b/tests/checkasm/vp9dsp.c index b9d1c73..690e0cf 10064

[libav-devel] [PATCH] aarch64: vp9itxfm: Use w3 instead of x3 for the int eob parameter

2016-11-18 Thread Martin Storsjö
The clobbering tests in checkasm are only invoked when testing correctness, so this bug didn't show up when benchmarking the dc-only version. --- libavcodec/aarch64/vp9itxfm_neon.S | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/aarch64/vp9itxfm_neon.S b/lib

[libav-devel] [PATCH 14/20] vima: Convert to the new bitstream reader

2016-11-18 Thread Diego Biurrun
From: Alexandra Hájková --- libavcodec/vima.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/libavcodec/vima.c b/libavcodec/vima.c index 6f539a8..0db1897 100644 --- a/libavcodec/vima.c +++ b/libavcodec/vima.c @@ -29,7 +29,7 @@ #include "adpcm_da

[libav-devel] [PATCH 19/20] ogg: Convert to the new bitstream reader

2016-11-18 Thread Diego Biurrun
From: Alexandra Hájková --- libavformat/oggparseflac.c | 22 -- libavformat/oggparsetheora.c | 39 +-- 2 files changed, 33 insertions(+), 28 deletions(-) diff --git a/libavformat/oggparseflac.c b/libavformat/oggparseflac.c index dab1040.

[libav-devel] [PATCH 09/20] truespeech: Convert to the new bitstream reader

2016-11-18 Thread Diego Biurrun
From: Alexandra Hájková --- libavcodec/truespeech.c | 88 + 1 file changed, 45 insertions(+), 43 deletions(-) diff --git a/libavcodec/truespeech.c b/libavcodec/truespeech.c index b2195ba..48e2126 100644 --- a/libavcodec/truespeech.c +++ b/libavcod

[libav-devel] [PATCH 15/20] wnv1: Convert to the new bitstream reader

2016-11-18 Thread Diego Biurrun
From: Alexandra Hájková --- libavcodec/wnv1.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libavcodec/wnv1.c b/libavcodec/wnv1.c index d0304c9..80d66ae 100644 --- a/libavcodec/wnv1.c +++ b/libavcodec/wnv1.c @@ -25,7 +25,7 @@ */ #include "avcodec.h" -#includ

[libav-devel] [PATCH 18/20] mpegts: Convert to the new bitstream reader

2016-11-18 Thread Diego Biurrun
From: Alexandra Hájková --- libavformat/mpegts.c | 48 +--- 1 file changed, 25 insertions(+), 23 deletions(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index 28b6d37..5024404 100644 --- a/libavformat/mpegts.c +++ b/libavformat/mpegts.c

[libav-devel] [PATCH 01/20] ra144: Convert to the new bitstream reader

2016-11-18 Thread Diego Biurrun
From: Alexandra Hájková --- libavcodec/ra144dec.c | 23 --- 1 file changed, 12 insertions(+), 11 deletions(-) diff --git a/libavcodec/ra144dec.c b/libavcodec/ra144dec.c index 2895357..35ee697 100644 --- a/libavcodec/ra144dec.c +++ b/libavcodec/ra144dec.c @@ -23,8 +23,9 @@

[libav-devel] [PATCH 07/20] truemotion2: Convert to the new bitstream reader

2016-11-18 Thread Diego Biurrun
From: Alexandra Hájková --- libavcodec/truemotion2.c | 40 1 file changed, 20 insertions(+), 20 deletions(-) diff --git a/libavcodec/truemotion2.c b/libavcodec/truemotion2.c index 1726440..99884be 100644 --- a/libavcodec/truemotion2.c +++ b/libavcodec/tr

[libav-devel] [PATCH 05/20] smacker: Convert to the new bitstream reader

2016-11-18 Thread Diego Biurrun
From: Alexandra Hájková --- libavcodec/smacker.c | 143 +++ 1 file changed, 75 insertions(+), 68 deletions(-) diff --git a/libavcodec/smacker.c b/libavcodec/smacker.c index e3e5475..027728e 100644 --- a/libavcodec/smacker.c +++ b/libavcodec/smacke

[libav-devel] [PATCH 08/20] tiertex: Convert to the new bitstream reader

2016-11-18 Thread Diego Biurrun
From: Alexandra Hájková --- libavcodec/tiertexseqv.c | 26 ++ 1 file changed, 14 insertions(+), 12 deletions(-) diff --git a/libavcodec/tiertexseqv.c b/libavcodec/tiertexseqv.c index 8ca7edf..d952266 100644 --- a/libavcodec/tiertexseqv.c +++ b/libavcodec/tiertexseqv.c @@

[libav-devel] [PATCH 20/20] rdt: Convert to the new bitstream reader

2016-11-18 Thread Diego Biurrun
From: Alexandra Hájková --- libavformat/rdt.c | 35 ++- 1 file changed, 18 insertions(+), 17 deletions(-) diff --git a/libavformat/rdt.c b/libavformat/rdt.c index eb718cf..da5ff59 100644 --- a/libavformat/rdt.c +++ b/libavformat/rdt.c @@ -25,6 +25,8 @@ * @autho

[libav-devel] [PATCH 13/20] vble: Convert to the new bitstream reader

2016-11-18 Thread Diego Biurrun
From: Alexandra Hájková --- libavcodec/vble.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/libavcodec/vble.c b/libavcodec/vble.c index 1a78036..fef1090 100644 --- a/libavcodec/vble.c +++ b/libavcodec/vble.c @@ -28,7 +28,7 @@ #define BITSTREA

[libav-devel] [PATCH 03/20] rtjpeg: Convert to the new bitstream reader

2016-11-18 Thread Diego Biurrun
From: Alexandra Hájková --- libavcodec/rtjpeg.c | 42 +++--- 1 file changed, 23 insertions(+), 19 deletions(-) diff --git a/libavcodec/rtjpeg.c b/libavcodec/rtjpeg.c index 67eeff8..baa1f78 100644 --- a/libavcodec/rtjpeg.c +++ b/libavcodec/rtjpeg.c @@ -18,8 +1

[libav-devel] [PATCH 06/20] svq1dec: Convert to the new bitstream reader

2016-11-18 Thread Diego Biurrun
From: Alexandra Hájková --- libavcodec/svq1dec.c | 120 +-- 1 file changed, 60 insertions(+), 60 deletions(-) diff --git a/libavcodec/svq1dec.c b/libavcodec/svq1dec.c index cc43f14..2ab0f23 100644 --- a/libavcodec/svq1dec.c +++ b/libavcodec/svq1de

[libav-devel] [PATCH 10/20] tscc2: Convert to the new bitstream reader

2016-11-18 Thread Diego Biurrun
From: Alexandra Hájková --- libavcodec/tscc2.c | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/libavcodec/tscc2.c b/libavcodec/tscc2.c index d3d45e4..d86428a 100644 --- a/libavcodec/tscc2.c +++ b/libavcodec/tscc2.c @@ -28,8 +28,8 @@ #define

[libav-devel] [PATCH 11/20] twinvq: Convert to the new bitstream reader

2016-11-18 Thread Diego Biurrun
From: Alexandra Hájková --- libavcodec/twinvq_data.h | 2 +- libavcodec/twinvqdec.c | 42 +- 2 files changed, 22 insertions(+), 22 deletions(-) diff --git a/libavcodec/twinvq_data.h b/libavcodec/twinvq_data.h index 01a54a5..cc7ba59 100644 --- a/libavco

[libav-devel] [PATCH 04/20] sipr: Convert to the new bitstream reader

2016-11-18 Thread Diego Biurrun
From: Alexandra Hájková --- libavcodec/sipr.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/libavcodec/sipr.c b/libavcodec/sipr.c index 686b3e2..cf29d3b 100644 --- a/libavcodec/sipr.c +++ b/libavcodec/sipr.c @@ -31,7 +31,7 @@ #define BITSTREAM_

[libav-devel] [PATCH 12/20] utvideodec: Convert to the new bitstream reader

2016-11-18 Thread Diego Biurrun
From: Alexandra Hájková --- libavcodec/utvideodec.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/libavcodec/utvideodec.c b/libavcodec/utvideodec.c index 3a58156..29de815 100644 --- a/libavcodec/utvideodec.c +++ b/libavcodec/utvideodec.c @@ -28,10 +28,11 @@

[libav-devel] [PATCH 16/20] xan: Convert to the new bitstream reader

2016-11-18 Thread Diego Biurrun
From: Alexandra Hájková --- libavcodec/xan.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/xan.c b/libavcodec/xan.c index c16c651..33149e5 100644 --- a/libavcodec/xan.c +++ b/libavcodec/xan.c @@ -37,8 +37,8 @@ #define BITSTREAM_READER_LE #include "avc

[libav-devel] [PATCH 17/20] xsubdec: Convert to the new bitstream reader

2016-11-18 Thread Diego Biurrun
From: Alexandra Hájková --- libavcodec/xsubdec.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/libavcodec/xsubdec.c b/libavcodec/xsubdec.c index 3af300c..a07f94c 100644 --- a/libavcodec/xsubdec.c +++ b/libavcodec/xsubdec.c @@ -21,8 +21,9 @@ #include "liba

[libav-devel] [PATCH 02/20] ra288: Convert to the new bitstream reader

2016-11-18 Thread Diego Biurrun
From: Alexandra Hájková --- libavcodec/ra288.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libavcodec/ra288.c b/libavcodec/ra288.c index 96d8d1e..bc3fe29 100644 --- a/libavcodec/ra288.c +++ b/libavcodec/ra288.c @@ -25,8 +25,8 @@ #define BITSTREAM_READER_LE

[libav-devel] new bitstream reader, second batch

2016-11-18 Thread Diego Biurrun
These patches have previously been approved on the mailing list. Will push after the weekend if there are no objections. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH] hlsenc: Fix the openssl support

2016-11-18 Thread Diego Biurrun
On Fri, Nov 18, 2016 at 10:57:23AM +0100, Luca Barbato wrote: > --- > > Thanks to Diego for spotting it. > > libavformat/hlsenc.c | 1 + > 1 file changed, 1 insertion(+) OK - and please push right away to fix the current breakage. Diego ___ libav-dev

Re: [libav-devel] [PATCH 2/3] ac3enc: Restructure DEBUG ifdefs to avoid unused function parameter warnings

2016-11-18 Thread Luca Barbato
On 18/11/2016 10:46, Diego Biurrun wrote: > +#define dprint_options(x) do {} while(0) or some compiler would complain about having ;; I'm afraid. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH] hlsenc: Fix the openssl support

2016-11-18 Thread Luca Barbato
--- Thanks to Diego for spotting it. libavformat/hlsenc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index 5eebec8..625531b 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/hlsenc.c @@ -92,6 +92,7 @@ static int randomize(uint8_t *buf, int l

[libav-devel] [PATCH 2/3] ac3enc: Restructure DEBUG ifdefs to avoid unused function parameter warnings

2016-11-18 Thread Diego Biurrun
--- libavcodec/ac3enc.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/libavcodec/ac3enc.c b/libavcodec/ac3enc.c index da141e1..bf74e74 100644 --- a/libavcodec/ac3enc.c +++ b/libavcodec/ac3enc.c @@ -1676,9 +1676,9 @@ void ff_ac3_output_frame(AC3EncodeContext *s, unsigned

[libav-devel] [PATCH 3/3] als: Restructure DEBUG ifdefs to avoid unused function parameter warnings

2016-11-18 Thread Diego Biurrun
--- libavcodec/alsdec.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/libavcodec/alsdec.c b/libavcodec/alsdec.c index f356a70..71b8450 100644 --- a/libavcodec/alsdec.c +++ b/libavcodec/alsdec.c @@ -246,9 +246,9 @@ typedef struct ALSBlockData { } ALSBlockData; +#ifd

[libav-devel] [PATCH 1/3] ape: Restructure DEBUG ifdefs to avoid unused function parameter warnings

2016-11-18 Thread Diego Biurrun
--- libavformat/ape.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/libavformat/ape.c b/libavformat/ape.c index 80e3372..d917815 100644 --- a/libavformat/ape.c +++ b/libavformat/ape.c @@ -94,9 +94,9 @@ static int ape_probe(AVProbeData * p) return 0; } +#ifdef DE

[libav-devel] [PATCH 1/2] arm: Add cbz/cbnz macros for arm mode

2016-11-18 Thread Martin Storsjö
This allows using the cbz/cbnz instruction in thumb mode, for the cases where the branch target is close enough. --- libavutil/arm/asm.S | 12 1 file changed, 12 insertions(+) diff --git a/libavutil/arm/asm.S b/libavutil/arm/asm.S index 4ac0ea2..e66c34c 100644 --- a/libavutil/arm/asm

[libav-devel] [PATCH 2/2] arm: vp9itxfm: Use cbz for a short jump forward

2016-11-18 Thread Martin Storsjö
--- The uses of cmp rX, #0, beq in the vp9 loop filter can't be converted to cbz, since the branch targets are too far away. --- libavcodec/arm/vp9itxfm_neon.S | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/libavcodec/arm/vp9itxfm_neon.S b/libavcodec/arm/vp9itxfm_neon.S inde