Re: [FFmpeg-devel] [PATCH 1/2] avcodec/mips: [loongson] Fixed mmi optimization

2020-09-10 Thread Michael Niedermayer
On Thu, Sep 03, 2020 at 02:29:51PM +0800, Shiyou Yin wrote:
> From: gxw 
> 
> Test case fate-checkasm-h264pred failed in latest community code.
> This patch fixed the bug.

will apply

thx

[...]
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

"I am not trying to be anyone's saviour, I'm trying to think about the
 future and not be sad" - Elon Musk



signature.asc
Description: PGP signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/mips: [loongson] Fixed mmi optimization

2020-09-10 Thread Shiyou Yin
>-Original Message-
>From: ffmpeg-devel-boun...@ffmpeg.org [mailto:ffmpeg-devel-boun...@ffmpeg.org] 
>On Behalf Of
>Shiyou Yin
>Sent: Thursday, September 3, 2020 2:30 PM
>To: ffmpeg-devel@ffmpeg.org
>Cc: gxw
>Subject: [FFmpeg-devel] [PATCH 1/2] avcodec/mips: [loongson] Fixed mmi 
>optimization
>
>From: gxw 
>
>Test case fate-checkasm-h264pred failed in latest community code.
>This patch fixed the bug.
>
>Signed-off-by: Shiyou Yin 
>---
> libavcodec/mips/h264pred_mmi.c | 8 ++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
>diff --git a/libavcodec/mips/h264pred_mmi.c b/libavcodec/mips/h264pred_mmi.c
>index f4fe091..0209c2e 100644
>--- a/libavcodec/mips/h264pred_mmi.c
>+++ b/libavcodec/mips/h264pred_mmi.c
>@@ -178,7 +178,9 @@ void ff_pred8x8l_top_dc_8_mmi(uint8_t *src, int 
>has_topleft,
>
> "1: \n\t"
> "bnez   %[has_topright],2f  \n\t"
>-"pinsrh_3   %[ftmp2],   %[ftmp2],   %[ftmp4]\n\t"
>+"dli%[tmp0],0xa4\n\t"
>+"mtc1   %[tmp0],%[ftmp1]\n\t"
>+"pshufh %[ftmp2],   %[ftmp2],   %[ftmp1]\n\t"
>
> "2: \n\t"
> "dli%[tmp0],0x02\n\t"
>@@ -370,7 +372,9 @@ void ff_pred8x8l_vertical_8_mmi(uint8_t *src, int 
>has_topleft,
>
> "1: \n\t"
> "bnez   %[has_topright],2f  \n\t"
>-"pinsrh_3   %[ftmp11],  %[ftmp11],  %[ftmp9]\n\t"
>+"dli%[tmp0],0xa4\n\t"
>+"mtc1   %[tmp0],%[ftmp1]\n\t"
>+"pshufh %[ftmp11],  %[ftmp11],  %[ftmp1]\n\t"
>
> "2: \n\t"
> "dli%[tmp0],0x02\n\t"
>--
>2.1.0
>

Ping.

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

[FFmpeg-devel] [PATCH 1/2] avcodec/mips: [loongson] Fixed mmi optimization

2020-09-02 Thread Shiyou Yin
From: gxw 

Test case fate-checkasm-h264pred failed in latest community code.
This patch fixed the bug.

Signed-off-by: Shiyou Yin 
---
 libavcodec/mips/h264pred_mmi.c | 8 ++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/libavcodec/mips/h264pred_mmi.c b/libavcodec/mips/h264pred_mmi.c
index f4fe091..0209c2e 100644
--- a/libavcodec/mips/h264pred_mmi.c
+++ b/libavcodec/mips/h264pred_mmi.c
@@ -178,7 +178,9 @@ void ff_pred8x8l_top_dc_8_mmi(uint8_t *src, int has_topleft,
 
 "1: \n\t"
 "bnez   %[has_topright],2f  \n\t"
-"pinsrh_3   %[ftmp2],   %[ftmp2],   %[ftmp4]\n\t"
+"dli%[tmp0],0xa4\n\t"
+"mtc1   %[tmp0],%[ftmp1]\n\t"
+"pshufh %[ftmp2],   %[ftmp2],   %[ftmp1]\n\t"
 
 "2: \n\t"
 "dli%[tmp0],0x02\n\t"
@@ -370,7 +372,9 @@ void ff_pred8x8l_vertical_8_mmi(uint8_t *src, int 
has_topleft,
 
 "1: \n\t"
 "bnez   %[has_topright],2f  \n\t"
-"pinsrh_3   %[ftmp11],  %[ftmp11],  %[ftmp9]\n\t"
+"dli%[tmp0],0xa4\n\t"
+"mtc1   %[tmp0],%[ftmp1]\n\t"
+"pshufh %[ftmp11],  %[ftmp11],  %[ftmp1]\n\t"
 
 "2: \n\t"
 "dli%[tmp0],0x02\n\t"
-- 
2.1.0

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".