[FFmpeg-devel] [PATCH] pixblockdsp: disable altivec optimizations on ppc64be
andreas.cadhalpun at googlemail.com
Wed Nov 2 22:32:01 EET 2016
On 02.11.2016 11:31, Michael Niedermayer wrote:
> On Wed, Nov 02, 2016 at 01:40:01AM +0100, Carl Eugen Hoyos wrote:
>> 2016-11-01 0:27 GMT+01:00 Andreas Cadhalpun <andreas.cadhalpun at googlemail.com>:
>>> -#if HAVE_ALTIVEC
>>> +#if HAVE_ALTIVEC && !(ARCH_PPC64 && HAVE_BIGENDIAN)
>>> if (!PPC_ALTIVEC(av_get_cpu_flags()))
>> Do the dnxhd tests work for you on 32bit BE powerpc?
Not when using the altivec functions.
(It's not tested on the Debian buildd, since the baseline of Debian's
powerpc architecture does not include altivec.)
>> (It fails here afaict)
> i find it strange to belive that the code never worked on either HW
Your intuition is correct. ;-)
> if we assume it did once work for some case, what is differnt now ?
> the testcase?
It wasn't tested with FATE, so it wasn't noticed, when it broke.
> the compiler?
> the code?
Yes, by the following Libav commit:
4c387c7 ppc: dsputil: do unaligned block accesses correctly
I'll send a patch fixing this in a moment.
More information about the ffmpeg-devel