Re: [FFmpeg-devel] [PATCH] avcodec/ppc/hevcdsp: Fix build failures with powerpc-linux-gnu-gcc-4.8 with --disable-optimizations

2018-12-05 Thread Michael Niedermayer
On Wed, Dec 05, 2018 at 12:15:14AM +0100, Carl Eugen Hoyos wrote: > 2018-12-04 18:02 GMT+01:00, Michael Niedermayer : > > On Tue, Dec 04, 2018 at 04:33:03PM +0100, Carl Eugen Hoyos wrote: > >> 2018-12-04 16:29 GMT+01:00, Michael Niedermayer : > >> > The affected functions could also be changed

Re: [FFmpeg-devel] [PATCH] avcodec/ppc/hevcdsp: Fix build failures with powerpc-linux-gnu-gcc-4.8 with --disable-optimizations

2018-12-04 Thread Carl Eugen Hoyos
2018-12-04 18:02 GMT+01:00, Michael Niedermayer : > On Tue, Dec 04, 2018 at 04:33:03PM +0100, Carl Eugen Hoyos wrote: >> 2018-12-04 16:29 GMT+01:00, Michael Niedermayer : >> > The affected functions could also be changed into macros, this is the >> > smaller change to fix it though. And avoids

Re: [FFmpeg-devel] [PATCH] avcodec/ppc/hevcdsp: Fix build failures with powerpc-linux-gnu-gcc-4.8 with --disable-optimizations

2018-12-04 Thread Michael Niedermayer
On Tue, Dec 04, 2018 at 04:33:03PM +0100, Carl Eugen Hoyos wrote: > 2018-12-04 16:29 GMT+01:00, Michael Niedermayer : > > The affected functions could also be changed into macros, this is the > > smaller change to fix it though. And avoids (probably) less readable macros > > > The extra code

Re: [FFmpeg-devel] [PATCH] avcodec/ppc/hevcdsp: Fix build failures with powerpc-linux-gnu-gcc-4.8 with --disable-optimizations

2018-12-04 Thread Carl Eugen Hoyos
2018-12-04 16:29 GMT+01:00, Michael Niedermayer : > The affected functions could also be changed into macros, this is the > smaller change to fix it though. And avoids (probably) less readable macros > The extra code should be optimized out when optimizations are done as all > values are known at

[FFmpeg-devel] [PATCH] avcodec/ppc/hevcdsp: Fix build failures with powerpc-linux-gnu-gcc-4.8 with --disable-optimizations

2018-12-04 Thread Michael Niedermayer
The affected functions could also be changed into macros, this is the smaller change to fix it though. And avoids (probably) less readable macros The extra code should be optimized out when optimizations are done as all values are known at build after inlining. Signed-off-by: Michael Niedermayer