[FFmpeg-devel] [PATCH 4/5] ppc: lavc: use LOCAL_ALIGNED instead of DECLARE_ALIGNED
James Almer
jamrial at gmail.com
Sun Mar 15 02:29:44 CET 2015
On 14/03/15 10:05 PM, Michael Niedermayer wrote:
> On Sat, Mar 14, 2015 at 08:35:29PM -0300, James Almer wrote:
>> On 14/03/15 3:18 PM, Michael Niedermayer wrote:
>>> On Sat, Mar 14, 2015 at 03:30:27PM +0000, Christophe Gisquet wrote:
>>>> The later may yield incorrect code for on-stack variables.
>>>> ---
>>>> libavcodec/ppc/h264dsp.c | 10 ++++-----
>>>> libavcodec/ppc/h264qpel.c | 50 ++++++++++++++++++++---------------------
>>>> libavcodec/ppc/vp8dsp_altivec.c | 2 +-
>>>> 3 files changed, 31 insertions(+), 31 deletions(-)
>>>
>>> applied
>>>
>>> thanks
>>
>> This apparently broke compilation for ppc.
>> http://fate.ffmpeg.org/report.cgi?time=20150314231437&slot=powerpc-linux-gnu-gcc-4.3.5
>>
>> /home/fate/fate/slots/powerpc-linux-gnu-gcc-4.3.5/src/libavcodec/ppc/h264dsp.c:270: error: invalid parameter combination for AltiVec intrinsic
>
> argh, i tested this, but i had altivec disabled it seems
>
>
>>
>> It should use LOCAL_ALIGNED_16() rather than LOCAL_ALIGNED(16,...), which shows that
>> whatever these LOCAL_ALIGNED_# macros do is needed, and the fifth patch in this set
>> is not correct.
>
> LOCAL_ALIGNED_# does not work either, ill revert the patch
>
> thanks
Ah, I see. I assumed it would since those macros are exclusively for x86 and ppc, thus
explaining why they exist in the first place.
I retract my comment about the fifth patch then.
More information about the ffmpeg-devel
mailing list