[FFmpeg-devel] [PATCH] do not pointlessly use REG_BP

Michael Niedermayer michaelni
Fri Jan 1 01:32:50 CET 2010


On Thu, Dec 31, 2009 at 10:34:35PM +0100, Reimar D?ffinger wrote:
> Hello,
> this might cause compilation issues, since EBP might not be available
> even on x86_64.
> Since there are enough register, this patch just lets gcc choose a
> register:
> Index: swscale_template.c
> ===================================================================
> --- swscale_template.c  (revision 30151)
> +++ swscale_template.c  (working copy)
> @@ -1225,18 +1225,19 @@
>          case PIX_FMT_RGB32:
>              if (CONFIG_SWSCALE_ALPHA && c->alpPixBuf) {
>  #if ARCH_X86_64
> +                x86_reg tmp;
>                  __asm__ volatile(
> -                    YSCALEYUV2RGB(%%REGBP, %5)
> -                    YSCALEYUV2RGB_YA(%%REGBP, %5, %6, %7)
> +                    YSCALEYUV2RGB(%0, %6)
> +                    YSCALEYUV2RGB_YA(%0, %6, %7, %8)
>                      "psraw                  $3, %%mm1       \n\t" /* abuf0[eax] - abuf1[eax] >>7*/
>                      "psraw                  $3, %%mm7       \n\t" /* abuf0[eax] - abuf1[eax] >>7*/
>                      "packuswb            %%mm7, %%mm1       \n\t"
> -                    WRITEBGR32(%4, 8280(%5), %%REGBP, %%mm2, %%mm4, %%mm5, %%mm1, %%mm0, %%mm7, %%mm3, %%mm6)
> +                    WRITEBGR32(%5, 8280(%6), %%REGBP, %%mm2, %%mm4, %%mm5, %%mm1, %%mm0, %%mm7, %%mm3, %%mm6)
                                                ^^^^^^^
and please test it next time

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

Incandescent light bulbs waste a lot of energy as heat so the EU forbids them.
Their replacement, compact fluorescent lamps, much more expensive, dont fit in
many old lamps, flicker, contain toxic mercury, produce a fraction of the light
that is claimed and in a unnatural spectrum rendering colors different than
in natural light. Ah and we now need to turn the heaters up more in winter to
compensate the lower wasted heat. Who wins? Not the environment, thats for sure
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100101/bbf40caa/attachment.pgp>



More information about the ffmpeg-devel mailing list