[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