Message ID | 1599114592-21832-1-git-send-email-yinshiyou-hf@loongson.cn |
---|---|
State | Accepted |
Commit | a4f7b09536c1d4b643992c03fa7105d2229da28e |
Headers | show |
Series | [FFmpeg-devel,1/2] avcodec/mips: [loongson] Fixed mmi optimization | expand |
Context | Check | Description |
---|---|---|
andriy/default | pending | |
andriy/make | success | Make finished |
andriy/make_fate | success | Make fate finished |
>-----Original Message----- >From: ffmpeg-devel-bounces@ffmpeg.org [mailto:ffmpeg-devel-bounces@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 <guxiwei-hf@loongson.cn> > >Test case fate-checkasm-h264pred failed in latest community code. >This patch fixed the bug. > >Signed-off-by: Shiyou Yin <yinshiyou-hf@loongson.cn> >--- > 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.
On Thu, Sep 03, 2020 at 02:29:51PM +0800, Shiyou Yin wrote: > From: gxw <guxiwei-hf@loongson.cn> > > Test case fate-checkasm-h264pred failed in latest community code. > This patch fixed the bug. will apply thx [...]
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"