[FFmpeg-devel] [PATCH] swscale.c
Reimar Döffinger
Reimar.Doeffinger
Thu Jun 4 10:40:06 CEST 2009
On Wed, Jun 03, 2009 at 08:19:14PM -0400, Pavel Pavlov wrote:
> > -----Original Message-----
> > On Wed, Jun 3, 2009 at 2:56 PM, Pavel Pavlov
> > <pavel at summit-tech.ca> wrote:
> > > Old:
> > > ? ? ? ?static const uint8_t ?__attribute__((aligned(8)))
> > > dither_2x2_4[2][8]={
> > >
> > > New:
> > > ? ? ? ?DECLARE_ALIGNED(8, static const uint8_t,
> > dither_2x2_4[2][8])={
> >
> > Hmm, shouldn't some of these (also in your other patches) use
> > DECLARE_ASM_CONST?
> >
> > Also is it best to:
> > DECLARE_ALIGNED(8, static const
> > static const DECLARE_ALIGNED(8
> > or some other combination?
> >
> > Some grepping shows we have a couple of different
> > combinations for those.
>
> I simply prefer to use one macro everywhere, instead of ten others.
> DECLARE_ASM_CONST just makes it less clear; with DECLARE_ALIGNED
> it's obvious that alignment is required and that variable is static or
> not.
If you don't use DECLARE_ASM_CONST for "static const" stuff that is used
via MANGLE in asm code you will break ICC compilation.
More information about the ffmpeg-devel
mailing list