diff mbox series

[FFmpeg-devel] avcodec/x86/fpel: Remove remnants of MMX

Message ID AS8P250MB0744EA68BE97D60800D4145D8F5F2@AS8P250MB0744.EURP250.PROD.OUTLOOK.COM
State Accepted
Commit a172d80703cd7779a58e9b671886a8c5383ce4d1
Headers show
Series [FFmpeg-devel] avcodec/x86/fpel: Remove remnants of MMX | expand

Checks

Context Check Description
yinshiyou/make_loongarch64 success Make finished
yinshiyou/make_fate_loongarch64 success Make fate finished
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Andreas Rheinhardt Feb. 29, 2024, 1:22 a.m. UTC
Since 7cad4dba505f9ad0adf013dd5bdf02a57249eecc
averaging functions only exist for MMXEXT and SSE2.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 libavcodec/x86/fpel.asm | 23 -----------------------
 1 file changed, 23 deletions(-)

Comments

Andreas Rheinhardt March 2, 2024, 1:54 a.m. UTC | #1
Andreas Rheinhardt:
> Since 7cad4dba505f9ad0adf013dd5bdf02a57249eecc
> averaging functions only exist for MMXEXT and SSE2.
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
> ---
>  libavcodec/x86/fpel.asm | 23 -----------------------
>  1 file changed, 23 deletions(-)
> 
> diff --git a/libavcodec/x86/fpel.asm b/libavcodec/x86/fpel.asm
> index 278d1410fc..b07b789074 100644
> --- a/libavcodec/x86/fpel.asm
> +++ b/libavcodec/x86/fpel.asm
> @@ -25,16 +25,6 @@
>  
>  SECTION .text
>  
> -%macro PAVGB_MMX 4
> -    LOAD   %3, %1
> -    por    %3, %2
> -    pxor   %2, %1
> -    pand   %2, %4
> -    psrlq  %2, 1
> -    psubb  %3, %2
> -    SWAP   %2, %3
> -%endmacro
> -
>  ; void ff_put/avg_pixels(uint8_t *block, const uint8_t *pixels,
>  ;                        ptrdiff_t line_size, int h)
>  %macro OP_PIXELS 2
> @@ -49,12 +39,6 @@ SECTION .text
>  %endif
>  cglobal %1_pixels%2, 4,5,4
>      lea          r4, [r2*3]
> -%ifidn %1, avg
> -%if notcpuflag(mmxext)
> -    pcmpeqd      m6, m6
> -    paddb        m6, m6
> -%endif
> -%endif
>  .loop:
>  %assign %%i 0
>  %rep LEN/mmsize
> @@ -63,17 +47,10 @@ cglobal %1_pixels%2, 4,5,4
>      LOAD         m2, [r1+r2*2 + %%i]
>      LOAD         m3, [r1+r4 + %%i]
>  %ifidn %1, avg
> -%if notcpuflag(mmxext)
> -    PAVGB_MMX    [r0 + %%i], m0, m4, m6
> -    PAVGB_MMX    [r0+r2 + %%i], m1, m5, m6
> -    PAVGB_MMX    [r0+r2*2 + %%i], m2, m4, m6
> -    PAVGB_MMX    [r0+r4 + %%i], m3, m5, m6
> -%else
>      pavgb        m0, [r0 + %%i]
>      pavgb        m1, [r0+r2 + %%i]
>      pavgb        m2, [r0+r2*2 + %%i]
>      pavgb        m3, [r0+r4 + %%i]
> -%endif
>  %endif
>      SAVE       [r0 + %%i], m0
>      SAVE    [r0+r2 + %%i], m1

Will apply this patch tomorrow unless there are objections.

- Andreas
diff mbox series

Patch

diff --git a/libavcodec/x86/fpel.asm b/libavcodec/x86/fpel.asm
index 278d1410fc..b07b789074 100644
--- a/libavcodec/x86/fpel.asm
+++ b/libavcodec/x86/fpel.asm
@@ -25,16 +25,6 @@ 
 
 SECTION .text
 
-%macro PAVGB_MMX 4
-    LOAD   %3, %1
-    por    %3, %2
-    pxor   %2, %1
-    pand   %2, %4
-    psrlq  %2, 1
-    psubb  %3, %2
-    SWAP   %2, %3
-%endmacro
-
 ; void ff_put/avg_pixels(uint8_t *block, const uint8_t *pixels,
 ;                        ptrdiff_t line_size, int h)
 %macro OP_PIXELS 2
@@ -49,12 +39,6 @@  SECTION .text
 %endif
 cglobal %1_pixels%2, 4,5,4
     lea          r4, [r2*3]
-%ifidn %1, avg
-%if notcpuflag(mmxext)
-    pcmpeqd      m6, m6
-    paddb        m6, m6
-%endif
-%endif
 .loop:
 %assign %%i 0
 %rep LEN/mmsize
@@ -63,17 +47,10 @@  cglobal %1_pixels%2, 4,5,4
     LOAD         m2, [r1+r2*2 + %%i]
     LOAD         m3, [r1+r4 + %%i]
 %ifidn %1, avg
-%if notcpuflag(mmxext)
-    PAVGB_MMX    [r0 + %%i], m0, m4, m6
-    PAVGB_MMX    [r0+r2 + %%i], m1, m5, m6
-    PAVGB_MMX    [r0+r2*2 + %%i], m2, m4, m6
-    PAVGB_MMX    [r0+r4 + %%i], m3, m5, m6
-%else
     pavgb        m0, [r0 + %%i]
     pavgb        m1, [r0+r2 + %%i]
     pavgb        m2, [r0+r2*2 + %%i]
     pavgb        m3, [r0+r4 + %%i]
-%endif
 %endif
     SAVE       [r0 + %%i], m0
     SAVE    [r0+r2 + %%i], m1