[FFmpeg-devel] [PATCH] remove redundant ARCH_ preprocessor checks

Diego Biurrun diego
Tue Mar 18 23:15:29 CET 2008


On Tue, Mar 18, 2008 at 02:45:37PM +0100, Diego Biurrun wrote:
> On Sat, Mar 15, 2008 at 02:53:59PM +0100, Aurelien Jacobs wrote:
> > On Sat, 15 Mar 2008 11:32:49 +0100
> > Diego Biurrun <diego at biurrun.de> wrote:
> > 
> > > I found a couple of redundant #if ARCH_* checks that enclose things like
> > > HAVE_ALTIVEC and similar more specific checks.  Attached is a patch to
> > > remove them.  I will apply this after the weekend unless I hear
> > > objections.
> > > 
> > > [...]
> > > 
> > > -#if defined(ARCH_X86)
> > > -
> > >  #if (defined (HAVE_MMX) && !defined (HAVE_3DNOW) && !defined (HAVE_MMX2)) || defined (RUNTIME_CPUDETECT)
> > >  #define COMPILE_MMX
> > >  #endif
> > 
> > I think removing this one is wrong, because RUNTIME_CPUDETECT can be
> > defined for non-x86 cpus.
> 
> Yes, thanks for reviewing this.  I'll apply a version without this
> hunk.

Applied.

Diego




More information about the ffmpeg-devel mailing list