[FFmpeg-devel] [PATCH] HACK: fix compilation with NASM.
Reimar Döffinger
Reimar.Doeffinger at gmx.de
Sun Nov 20 12:45:19 CET 2011
On Sat, Nov 19, 2011 at 01:17:22PM +0900, KO Myung-Hun wrote:
>
>
> Reimar Döffinger wrote:
> > This issue is actually already fixed in NASM, however it will be a
> > bit before it arrives everywhere.
> > This should work as a temporary work-around.
> >
> > Signed-off-by: Reimar Döffinger <Reimar.Doeffinger at gmx.de>
> > ---
> > libavutil/x86/x86inc.asm | 15 +++++++++++++++
> > 1 files changed, 15 insertions(+), 0 deletions(-)
> >
> > diff --git a/libavutil/x86/x86inc.asm b/libavutil/x86/x86inc.asm
> > index 842a2cc..4e1eef5 100644
> > --- a/libavutil/x86/x86inc.asm
> > +++ b/libavutil/x86/x86inc.asm
> > @@ -468,9 +468,19 @@ DECLARE_REG 6, ebp, ebp, bp, null, [esp + stack_offset + 28]
> > ; Appends cpuflags to the function name if cpuflags has been specified.
> > %macro cglobal 1-2+ ; name, [PROLOGUE args]
> > %if %0 == 1
> > + ; HACK: work around %+ broken with empty SUFFIX for nasm 2.09.10
> > + %ifempty SUFFIX
> > + cglobal_internal %1
> > + %else
> > cglobal_internal %1 %+ SUFFIX
> > + %endif
> > %else
> > + ; HACK: work around %+ broken with empty SUFFIX for nasm 2.09.10
> > + %ifempty SUFFIX
> > + cglobal_internal %1, %2
> > + %else
> > cglobal_internal %1 %+ SUFFIX, %2
> > + %endif
> > %endif
> > %endmacro
> > %macro cglobal_internal 1-2+
> > @@ -747,7 +757,12 @@ INIT_XMM
> >
> > ; Append cpuflags to the callee's name iff the appended name is known and the plain name isn't
> > %macro call 1
> > + ; HACK: work around %+ broken with empty SUFFIX for nasm 2.09.10
> > + %ifempty SUFFIX
> > + call_internal %1, %1
> > + %else
> > call_internal %1, %1 %+ SUFFIX
> > + %endif
> > %endmacro
> > %macro call_internal 2
> > %xdefine %%i %1
>
> ok with me.
Pushed, it's easy enough to remove and I'm tired of having it locally
(I don't have yasm installed on purpose to motivate me to keep nasm
working :-) ).
More information about the ffmpeg-devel
mailing list