From: Andreas Rheinhardt <andreas.rheinhardt@outlook.com> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org> Subject: [FFmpeg-devel] [PATCH] tests/checkasm/videodsp: Don't use declare_func_emms Date: Wed, 19 Mar 2025 11:48:30 +0100 Message-ID: <AS8P250MB0744B21E88B1B80F5466327A8FD92@AS8P250MB0744.EURP250.PROD.OUTLOOK.COM> (raw) [-- Attachment #1: Type: text/plain, Size: 27 bytes --] Patch attached. - Andreas [-- Attachment #2: 0001-tests-checkasm-videodsp-Don-t-use-declare_func_emms.patch --] [-- Type: text/x-patch, Size: 2042 bytes --] From 69cc1770876b15331a448138f77866990ad4b0d0 Mon Sep 17 00:00:00 2001 From: Andreas Rheinhardt <andreas.rheinhardt@outlook.com> Date: Wed, 19 Mar 2025 11:42:10 +0100 Subject: [PATCH] tests/checkasm/videodsp: Don't use declare_func_emms It allows the callee to clobber the MMX state, yet since 1e3dc705dfb9546f0f7ff4479d20baeb4bdb6187 this is no longer done. So use the stricter declare_func instead. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com> --- tests/checkasm/videodsp.c | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/tests/checkasm/videodsp.c b/tests/checkasm/videodsp.c index eb2845a7d9..9751ec8658 100644 --- a/tests/checkasm/videodsp.c +++ b/tests/checkasm/videodsp.c @@ -66,13 +66,12 @@ do { \ LOCAL_ALIGNED_16(type, dst0, [64 * 64]); \ LOCAL_ALIGNED_16(type, dst1, [64 * 64]); \ - declare_func_emms(AV_CPU_FLAG_MMX | AV_CPU_FLAG_MMXEXT, \ - void, type *dst, const type *src, \ - ptrdiff_t dst_linesize, \ - ptrdiff_t src_linesize, \ - int block_w, int block_h, \ - int src_x, int src_y, \ - int src_w, int src_h); \ + declare_func(void, type *dst, const type *src, \ + ptrdiff_t dst_linesize, \ + ptrdiff_t src_linesize, \ + int block_w, int block_h, \ + int src_x, int src_y, \ + int src_w, int src_h); \ check_emu_edge_size(type, 16, 1, 64, 64); \ check_emu_edge_size(type, 16, 16, 64, 64); \ check_emu_edge_size(type, 64, 64, 64, 64); \ -- 2.45.2 [-- Attachment #3: Type: text/plain, Size: 251 bytes --] _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
next reply other threads:[~2025-03-19 10:48 UTC|newest] Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top 2025-03-19 10:48 Andreas Rheinhardt [this message] 2025-03-21 3:28 ` Andreas Rheinhardt
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=AS8P250MB0744B21E88B1B80F5466327A8FD92@AS8P250MB0744.EURP250.PROD.OUTLOOK.COM \ --to=andreas.rheinhardt@outlook.com \ --cc=ffmpeg-devel@ffmpeg.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel This inbox may be cloned and mirrored by anyone: git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \ ffmpegdev@gitmailbox.com public-inbox-index ffmpegdev Example config snippet for mirrors. AGPL code for this site: git clone https://public-inbox.org/public-inbox.git