On Fri, Aug 21, 2015 at 02:20:20PM -0400, Ganesh Ajjanagadde wrote: > On Fri, Aug 21, 2015 at 2:17 PM, Ganesh Ajjanagadde > <gajjanaga...@gmail.com> wrote: > > > > REG_SP is defined by Solaris system headers. > > This fixes a sea of warnings while building on Solaris: > > http://fate.ffmpeg.org/report.cgi?time=20150820233505&slot=x86-opensolaris-gcc4.3 > > > > Signed-off-by: Ganesh Ajjanagadde <gajjanaga...@gmail.com> > > --- > > libavutil/x86/asm.h | 5 +++-- > > libpostproc/postprocess_template.c | 2 +- > > 2 files changed, 4 insertions(+), 3 deletions(-) > > > > diff --git a/libavutil/x86/asm.h b/libavutil/x86/asm.h > > index 616ad6c..109b65e 100644 > > --- a/libavutil/x86/asm.h > > +++ b/libavutil/x86/asm.h > > @@ -38,7 +38,8 @@ typedef struct ymm_reg { uint64_t a, b, c, d; } ymm_reg; > > # define PTR_SIZE "8" > > typedef int64_t x86_reg; > > > > -# define REG_SP "rsp" > > +/* REG_SP is defined in Solaris sys headers, so use REG_sp */ > > +# define REG_sp "rsp" > > # define REG_BP "rbp" > > # define REGBP rbp > > # define REGa rax > > @@ -59,7 +60,7 @@ typedef int64_t x86_reg; > > # define PTR_SIZE "4" > > typedef int32_t x86_reg; > > > > -# define REG_SP "esp" > > +# define REG_sp "esp" > > # define REG_BP "ebp" > > # define REGBP ebp > > # define REGa eax > > diff --git a/libpostproc/postprocess_template.c > > b/libpostproc/postprocess_template.c > > index 5694cc5..b01be58 100644 > > --- a/libpostproc/postprocess_template.c > > +++ b/libpostproc/postprocess_template.c > > @@ -1317,7 +1317,7 @@ DERING_CORE((%0, %1, 8) ,(%%REGd, %1, > > 4),%%mm2,%%mm4,%%mm0,%%mm3,%%mm5,%%mm1, > > "1: \n\t" > > : : "r" (src), "r" ((x86_reg)stride), "m" (c->pQPb), > > "m"(c->pQPb2), "q"(tmp) > > NAMED_CONSTRAINTS_ADD(deringThreshold,b00,b02,b08) > > - : "%"REG_a, "%"REG_d, "%"REG_SP > > + : "%"REG_a, "%"REG_d, "%"REG_sp > > ); > > #else // HAVE_7REGS && (TEMPLATE_PP_MMXEXT || TEMPLATE_PP_3DNOW) > > int y; > > -- > > 2.5.0 > > > > @Michael: Can you test this patch on Solaris, > assuming the michael in > http://fate.ffmpeg.org/report.cgi?time=20150820233505&slot=x86-opensolaris-gcc4.3 > is you? > I do not have Solaris with me, but based patch off of what I read online.
it seems to fix the issue [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Democracy is the form of government in which you can choose your dictator
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
