On Thu, Jul 23, 2020 at 04:18:32PM +0200, Michael Niedermayer wrote: > On Tue, Jul 21, 2020 at 03:40:25PM +0800, Shiyou Yin wrote: > > >-----Original Message----- > > >From: ffmpeg-devel-boun...@ffmpeg.org > > >[mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of > > >Jiaxun Yang > > >Sent: Saturday, July 18, 2020 11:36 PM > > >To: ffmpeg-devel@ffmpeg.org > > >Cc: Jiaxun Yang > > >Subject: [FFmpeg-devel] [PATCH v6 0/6] MIPS MSA & MMI Runtime detection > > >support > > > > > >This series adds MIPS MSA & MMI runtime detection support > > > > > >Please review. > > > > > >Thanks! > > > > > >v2: > > > - Add CPUCFG support. > > > - Add "-mloongson-ext" to MMIFLAGS for Loongson-3 as well. (Loongson2F > > > don't need this flag) > > > > > >v3: > > > - Address reveiew suggestions from Shiyou Yin and Weixi Gu. > > > > > >v4: > > > - Disable DSP for generic CPU > > > > > >v5: > > > - Clean ups > > > - Address some GCC build warnings > > > > > >v6: > > > - Address more Shiyou's comments > > > > > >Jiaxun Yang (6): > > > ffbuild: Refine MIPS handling > > > libavutils: Add parse_r helper for MIPS > > > libavutil: Detect MMI and MSA flags for MIPS > > > libavcodec: Enable runtime detection for MIPS MMI & MSA > > > libavcodec: MIPS: MMI: Fix type mismatches > > > libavcodec: MIPS: MMI: Move sp out of the clobber list > > > > > > configure | 172 ++-- > > > ffbuild/common.mak | 10 +- > > > libavcodec/mips/Makefile | 3 +- > > > libavcodec/mips/blockdsp_init_mips.c | 40 +- > > > libavcodec/mips/cabac.h | 2 +- > > > libavcodec/mips/h263dsp_init_mips.c | 18 +- > > > libavcodec/mips/h264chroma_init_mips.c | 55 +- > > > libavcodec/mips/h264dsp_init_mips.c | 225 +++-- > > > libavcodec/mips/h264dsp_mips.h | 18 +- > > > libavcodec/mips/h264dsp_mmi.c | 56 +- > > > libavcodec/mips/h264pred_init_mips.c | 207 ++-- > > > libavcodec/mips/h264qpel_init_mips.c | 412 ++++---- > > > libavcodec/mips/hevcdsp_init_mips.c | 992 ++++++++++---------- > > > libavcodec/mips/hevcpred_init_mips.c | 40 +- > > > libavcodec/mips/hpeldsp_init_mips.c | 180 ++-- > > > libavcodec/mips/idctdsp_init_mips.c | 74 +- > > > libavcodec/mips/me_cmp_init_mips.c | 50 +- > > > libavcodec/mips/mpegvideo_init_mips.c | 48 +- > > > libavcodec/mips/mpegvideoencdsp_init_mips.c | 21 +- > > > libavcodec/mips/pixblockdsp_init_mips.c | 63 +- > > > libavcodec/mips/qpeldsp_init_mips.c | 270 +++--- > > > libavcodec/mips/vc1dsp_init_mips.c | 186 ++-- > > > libavcodec/mips/videodsp_init.c | 10 +- > > > libavcodec/mips/vp3dsp_init_mips.c | 44 +- > > > libavcodec/mips/vp8dsp_init_mips.c | 240 +++-- > > > libavcodec/mips/vp9dsp_init_mips.c | 16 +- > > > libavcodec/mips/wmv2dsp_init_mips.c | 18 +- > > > libavcodec/mips/wmv2dsp_mips.h | 4 +- > > > libavcodec/mips/wmv2dsp_mmi.c | 4 +- > > > libavcodec/mips/xvid_idct_mmi.c | 4 +- > > > libavcodec/mips/xvididct_init_mips.c | 31 +- > > > libavcodec/mips/xvididct_mips.h | 4 +- > > > libavutil/cpu.c | 10 + > > > libavutil/cpu.h | 3 + > > > libavutil/cpu_internal.h | 2 + > > > libavutil/mips/Makefile | 2 +- > > > libavutil/mips/asmdefs.h | 42 + > > > libavutil/mips/cpu.c | 134 +++ > > > libavutil/mips/cpu.h | 28 + > > > libavutil/tests/cpu.c | 3 + > > > tests/checkasm/checkasm.c | 3 + > > > 41 files changed, 1919 insertions(+), 1825 deletions(-) > > > create mode 100644 libavutil/mips/cpu.c > > > create mode 100644 libavutil/mips/cpu.h > > > > > >-- > > >2.27.0 > > > > LGTM > > will apply
next time please make sure the patches do not add tabs in .h / .c files these cannot be pushed thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Old school: Use the lowest level language in which you can solve the problem conveniently. New school: Use the highest level language in which the latest supercomputer can solve the problem without the user falling asleep waiting.
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".