>-----Original Message----- >From: ffmpeg-devel-boun...@ffmpeg.org [mailto:ffmpeg-devel-boun...@ffmpeg.org] >On Behalf Of >Michael Niedermayer >Sent: Friday, August 31, 2018 5:47 AM >To: FFmpeg development discussions and patches >Subject: Re: [FFmpeg-devel] [PATCH v2] avcodec/mips: [loongson] reoptimize >h264_chroma_mc8_mmi v2. > >On Thu, Aug 30, 2018 at 02:24:04PM +0800, Shiyou Yin wrote: >> Reoptimize function ff_put_h264_chroma_mc8_mmi and >> ff_avg_h264_chroma_mc8_mmi. >> Performance of h264 decoding improved about 5%(from 69fps to 73fps, tested >> on loongson 3A3000). > >what do you mean by "Reoptimize"? >does this port some optimizations from elsewhere ? >does this take the same code as previous optimizations did and re implements >(better/faster) MIPS code based on it ? > >what is the speed difference ?
Two functions have optimized with mmi yet. This patch was based on the previous version, then optimized the branch condition and the code in branch. This patch will speed up about 5% for h264 decode on loongson platform. >> >> Change-Id: Iccd7f4e480b2d0bfc47e4d409874c4adb77416cc > >what is this ? The original patch was made from loongson's local repository(we use gerrit to manage patch review and merge), each commit has it's own Change-Id. When I use git am to apply this patch to ffmpeg source, so the Change-Id was keeped with commit message. I will remove it in the next version. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
