[FFmpeg-devel] [PATCH 2/2] postproc/postprocess_template: remove FF_REG_sp from clobber list

[Michael Niedermayer]

On Thu, Dec 20, 2018 at 06:54:12PM -0300, James Almer wrote:
> On 12/20/2018 6:40 PM, Michael Niedermayer wrote:
> > Future gcc may no longer support this
> > 
> > Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> > ---
> >  libpostproc/postprocess_template.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/libpostproc/postprocess_template.c b/libpostproc/postprocess_template.c
> > index 485eb5cfc0..b0adfd168c 100644
> > --- a/libpostproc/postprocess_template.c
> > +++ b/libpostproc/postprocess_template.c
> > @@ -1317,7 +1317,7 @@ DERING_CORE((%0, %1, 8)       ,(%%FF_REGd, %1, 4),%%mm2,%%mm4,%%mm0,%%mm3,%%mm5,
> >          "1:                        \n\t"
> >          : : "r" (src), "r" ((x86_reg)stride), "m" (c->pQPb), "m"(c->pQPb2), "q"(tmp)
> >            NAMED_CONSTRAINTS_ADD(deringThreshold,b00,b02,b08)
> > -        : "%"FF_REG_a, "%"FF_REG_d, "%"FF_REG_sp
> > +        : "%"FF_REG_a, "%"FF_REG_d
> >      );
> >  #else // HAVE_7REGS && (TEMPLATE_PP_MMXEXT || TEMPLATE_PP_3DNOW)
> >      int y;
> 
> The set compiles with gcc 9 now, so LGTM if tested. Should be backported
> to supported releases as well.

ive tested it locally on x86 32&64 but not with gcc 9, it would be ideal
if someone tests this with gcc 9 beyond compilation

this should use the code, the "31" can be varied to test different "strength"
-vf 'pp=de/fq|31'

thx

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

Whats the most studid thing your enemy could do ? Blow himself up
Whats the most studid thing you could do ? Give up your rights and
freedom because your enemy blew himself up.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20181221/af18af55/attachment.sig>