diff mbox series

[FFmpeg-devel,01/20] libpostproc/postprocess_template: Don't reimplement FFSWAP

Message ID AM7PR03MB6660647098FC21E8418ECA608FAB9@AM7PR03MB6660.eurprd03.prod.outlook.com
State Accepted
Commit 5d10e218097008e672078bcf871f36cf57d17908
Headers show
Series [FFmpeg-devel,01/20] libpostproc/postprocess_template: Don't reimplement FFSWAP | expand

Commit Message

Andreas Rheinhardt Oct. 1, 2021, 9:02 p.m. UTC 
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 libpostproc/postprocess_template.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

Comments

Paul B Mahol Oct. 2, 2021, 3:02 p.m. UTC | #1 
ok
Michael Niedermayer Oct. 2, 2021, 4:19 p.m. UTC | #2 
On Fri, Oct 01, 2021 at 11:02:02PM +0200, Andreas Rheinhardt wrote:
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
> ---
>  libpostproc/postprocess_template.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)

LGTM

[...]
diff mbox series

Patch

diff --git a/libpostproc/postprocess_template.c b/libpostproc/postprocess_template.c
index 411fc41915..9f76b7c587 100644
--- a/libpostproc/postprocess_template.c
+++ b/libpostproc/postprocess_template.c
@@ -3595,7 +3595,6 @@  static void RENAME(postProcess)(const uint8_t src[], int srcStride, uint8_t dst[
 
           for(x = startx, qp_index=0; x < endx; x+=BLOCK_SIZE, qp_index++){
             const int stride= dstStride;
-            av_unused uint8_t *tmpXchg;
             c.QP = c.QP_block[qp_index];
             c.nonBQP = c.nonBQP_block[qp_index];
             c.pQPb = c.pQPb_block[qp_index];
@@ -3668,9 +3667,7 @@  static void RENAME(postProcess)(const uint8_t src[], int srcStride, uint8_t dst[
             srcBlock+=8;
 
 #if TEMPLATE_PP_MMX
-            tmpXchg= tempBlock1;
-            tempBlock1= tempBlock2;
-            tempBlock2 = tmpXchg;
+            FFSWAP(uint8_t *, tempBlock1, tempBlock2);
 #endif
           }
         }