[FFmpeg-devel] [PATCH 1/2] mips: Add dependencies on HAVE_INLINE_ASM

Babic, Nedeljko nbabic at mips.com
Tue Nov 6 15:55:45 CET 2012


Hello!

Problem with ff_dsputil_init_mips() is just an example. 
Almost in all files there are similar init functions.

Anyway, I will send patch so it will be more clear what 
I meant.

-Nedeljko
________________________________________
From: Vitor Sessak [vitor1001 at gmail.com]
Sent: Saturday, November 03, 2012 18:33
To: FFmpeg development discussions and patches
Cc: Babic, Nedeljko; Lukac, Zeljko
Subject: Re: [FFmpeg-devel] [PATCH 1/2] mips: Add dependencies on HAVE_INLINE_ASM

H!

On 11/02/2012 02:39 PM, Babic, Nedeljko wrote:
> Hi and sorry for the late response.
>
> As my colleague mentioned, I was not working last more than two weeks...
>
> Regarding comments on patch:
>
> I wanted to make these files useful for all codecs that use them and since
> these entire files depend on HAVE_INLINE_ASM to compile them only in that case
> (like it was suggested for AC3 FP decoder patch), but I was mistaking...
>
> As Vitor noted, if this patch is applied, code would not compile correctly without
> HAVE_INLINE_ASM activated. The problem is that in those files appropriate init
> functions are defined and they are used in files that don't depend on
> HAVE_INLINE_ASM.
>
> One solution to this would be to make things dependant on A && B using
> intermediate in Makefile as suggested and to check if HAVE_INLINE_ASM is activated
> on places where functions are called. I am not sure if this is good solution because
> we would have for example:
>      ...
>      if (ARCH_SH4)        ff_dsputil_init_sh4   (c, avctx);
>      if (ARCH_BFIN)       ff_dsputil_init_bfin  (c, avctx);
>      if (HAVE_MIPSFPU && HAVE_INLINE_ASM)    ff_dsputil_init_mips  (c, avctx);
>
> instead of:
>      ...
>      if (ARCH_SH4)        ff_dsputil_init_sh4   (c, avctx);
>      if (ARCH_BFIN)       ff_dsputil_init_bfin  (c, avctx);
>      if (HAVE_MIPSFPU)    ff_dsputil_init_mips  (c, avctx);
>
> Instead of this I am thinking to move HAVE_INLINE_ASM dependencies back in files
> and to use #ifs in files for checking.

Another solution would be only adding #ifs in the file that contain
ff_dsputil_init_mips() and make the compilation of the other ones
conditional on HAVE_INLINE_ASM.

-Vitor


More information about the ffmpeg-devel mailing list