[FFmpeg-devel] [PATCH 2/4] x86/qpel: move fullpel and l2 functions to separate file.

Ronald S. Bultje rsbultje at gmail.com
Sat Mar 9 05:40:06 CET 2013


Hi,

On Fri, Mar 8, 2013 at 8:38 PM, Ronald S. Bultje <rsbultje at gmail.com> wrote:
> Hi,
>
> On Mon, Mar 4, 2013 at 3:13 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
>> On Sun, Mar 03, 2013 at 03:47:10PM -0800, Ronald S. Bultje wrote:
>>> From: "Ronald S. Bultje" <rsbultje at gmail.com>
>>>
>>> This way, they can be shared between mpeg4qpel and h264qpel without
>>> requiring either one to be compiled unconditionally.
>>> ---
>>>  libavcodec/x86/Makefile     |   4 +-
>>>  libavcodec/x86/dsputil.asm  | 192 ---------------------------------
>>>  libavcodec/x86/hpeldsp.asm  |  41 -------
>>>  libavcodec/x86/qpelbase.asm | 254 ++++++++++++++++++++++++++++++++++++++++++++
>>>  4 files changed, 257 insertions(+), 234 deletions(-)
>>>  create mode 100644 libavcodec/x86/qpelbase.asm
>>
>> doesnt build:
>> `ff_avg_pixels8_mmxext' referenced in section `.text' of libavcodec/libavcodec.a(dsputil_mmx.o): defined in discarded section `.note.GNU-stack' of libavcodec/libavcodec.a(qpelbase.o)
>> `ff_put_pixels8_mmxext' referenced in section `.text' of libavcodec/libavcodec.a(dsputil_mmx.o): defined in discarded section `.note.GNU-stack' of libavcodec/libavcodec.a(qpelbase.o)
>> `ff_put_pixels8_mmxext' referenced in section `.text' of libavcodec/libavcodec.a(dsputil_mmx.o): defined in discarded section `.note.GNU-stack' of libavcodec/libavcodec.a(qpelbase.o)
>> `ff_put_pixels8_l2_mmxext' referenced in section `.text' of libavcodec/libavcodec.a(dsputil_mmx.o): defined in discarded section `.note.GNU-stack' of libavcodec/libavcodec.a(qpelbase.o)
>> `ff_put_pixels8_l2_mmxext' referenced in section `.text' of libavcodec/libavcodec.a(dsputil_mmx.o): defined in discarded section `.note.GNU-stack' of libavcodec/libavcodec.a(qpelbase.o)
>> `ff_put_pixels8_l2_mmxext' referenced in section `.text' of libavcodec/libavcodec.a(dsputil_mmx.o): defined in discarded section `.note.GNU-stack' of libavcodec/libavcodec.a(qpelbase.o)
>> (above is a random sample of errors, its many pages of errors)
>>
>> with 1/4:
>> libavcodec/dsputil.c:1312:6: error: no previous prototype for ‘ff_put_pixels8x8’ [-Werror=missing-prototypes]
>> libavcodec/dsputil.c:1315:6: error: no previous prototype for ‘ff_avg_pixels8x8’ [-Werror=missing-prototypes]
>> libavcodec/dsputil.c:1318:6: error: no previous prototype for ‘ff_put_pixels16x16’ [-Werror=missing-prototypes]
>> libavcodec/dsputil.c:1321:6: error: no previous prototype for ‘ff_avg_pixels16x16’ [-Werror=missing-prototypes]
>
> Yeah OK I forgot to test 1/4, but I can't reproduce the first half of
> your errors...

Oh n/m forgot section .text...

Ronald


More information about the ffmpeg-devel mailing list