[Ffmpeg-devel] [PATCH] build with '--enable-gprof'

[Michael Niedermayer]

Hi

On Thu, Dec 07, 2006 at 06:49:52PM +0300, Dmitry Antipov wrote:
> M?ns Rullg?rd wrote:
> 
> >Dmitry Antipov said:
> >
> >>+    ebp_is_special="yes"
> >>+    check_cc <<EOF && ebp_is_special="no"
> >
> >check_cc <<EOF || enable ebp_is_special
> >
> >>+test "$ebp_is_special" = "yes" && echo '#define EBP_IS_SPECIAL 1' >> 
> >>$TMPH
> >
> >Instead of this, add ebp_is_special to CONFIG_LIST, and 
> >CONFIG_EBP_IS_SPECIAL will
> >be set automatically.
> >
> 
> Ok
> 
> Dmitry

[...]
> Index: libavcodec/cabac.h
> ===================================================================
> --- libavcodec/cabac.h	(revision 7245)
> +++ libavcodec/cabac.h	(working copy)
> @@ -376,7 +376,7 @@
>  #define BYTE        "16"
>  #define BYTEEND     "20"
>  #endif
> -#if defined(ARCH_X86) && !(defined(PIC) && defined(__GNUC__))
> +#if defined(ARCH_X86) && !defined(CONFIG_EBP_IS_SPECIAL)

this isnt correct its rather

#if defined(ARCH_X86) && !defined(CONFIG_EBP_IS_SPECIAL) && !defined(CONFIG_EBX_IS_SPECIAL)

as for PIC code with -fomit-frame-pointer EBP isnt special and the asm code
in configure will compile and work correctly still gcc will fail compiling
the asm above as EBX is special for gcc in the PIC case, 

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If you really think that XML is the answer, then you definitly missunderstood
the question -- Attila Kinali