On 01/14/2013 04:28 PM, Diego Biurrun wrote: > --- > > Added comments about macro arguments as requested by Justin. > > libavutil/x86/x86util.asm | 11 +++++------ > 1 files changed, 5 insertions(+), 6 deletions(-) > > diff --git a/libavutil/x86/x86util.asm b/libavutil/x86/x86util.asm > index 9300faf..4740b7f 100644 > --- a/libavutil/x86/x86util.asm > +++ b/libavutil/x86/x86util.asm > @@ -211,10 +211,14 @@ > %endif > %endmacro > > -%macro ABSB_MMX 2 > +%macro ABSB 2 ; source mmreg, temp mmreg (unused for ssse3) > +%if cpuflag(ssse3) > + pabsb %1, %1 > +%else > pxor %2, %2 > psubb %2, %1 > pminub %1, %2 > +%endif > %endmacro > > %macro ABSB2_MMX 4 > @@ -237,10 +241,6 @@ > psubd %2, %4 > %endmacro > > -%macro ABSB_SSSE3 2 > - pabsb %1, %1 > -%endmacro > - > %macro ABSB2_SSSE3 4 > pabsb %1, %1 > pabsb %2, %2 > @@ -251,7 +251,6 @@ > ABS2 %3, %4, %5, %6 > %endmacro > > -%define ABSB ABSB_MMX > %define ABSB2 ABSB2_MMX > > %macro SPLATB_LOAD 3
LGTM -Justin _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
