[FFmpeg-devel] [PATCH 4/9] x264asm: fix cmp* number of arguments

[Ronald S. Bultje]

Hi,


On Thu, Apr 4, 2013 at 12:45 PM, Christophe Gisquet <
christophe.gisquet at gmail.com> wrote:

> cmp{p,s}{s,d} instructions do take an imm8 operand.
> ---
>  libavutil/x86/x86inc.asm | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/libavutil/x86/x86inc.asm b/libavutil/x86/x86inc.asm
> index 068e3cf..636213b 100644
> --- a/libavutil/x86/x86inc.asm
> +++ b/libavutil/x86/x86inc.asm
> @@ -1114,10 +1114,10 @@ AVX_INSTR blendpd, 1, 0, 0
>  AVX_INSTR blendps, 1, 0, 0
>  AVX_INSTR blendvpd, 1, 0, 0
>  AVX_INSTR blendvps, 1, 0, 0
> -AVX_INSTR cmppd, 1, 0, 0
> -AVX_INSTR cmpps, 1, 0, 0
> -AVX_INSTR cmpsd, 1, 0, 0
> -AVX_INSTR cmpss, 1, 0, 0
> +AVX_INSTR cmppd, 1, 1, 0
> +AVX_INSTR cmpps, 1, 1, 0
> +AVX_INSTR cmpsd, 1, 1, 0
> +AVX_INSTR cmpss, 1, 1, 0
>  AVX_INSTR cvtdq2ps, 1, 0, 0
>  AVX_INSTR cvtpd2dq, 1, 0, 0
>  AVX_INSTR cvtps2dq, 1, 0, 0
>

Does x264 have this relevant patch in their tree also?

Ronald