From: "Rémi Denis-Courmont" <remi@remlab.net>
To: ffmpeg-devel@ffmpeg.org
Subject: [FFmpeg-devel] [PATCH 3/4] lavu/riscv: use Zbb CPOP/CPOPW at run-time
Date: Sat, 8 Jun 2024 14:37:15 +0300
Message-ID: <20240608113717.1677043-3-remi@remlab.net> (raw)
In-Reply-To: <20240608113717.1677043-1-remi@remlab.net>
Zbb static Zbb dynamic I baseline
popcount 1.336129286 3.469067758 20.146362909
popcountl 1.336322291 3.340292968 20.224829821
(seconds for 1 billion iterations on a SiFive-U74 core)
---
libavutil/riscv/intmath.h | 73 ++++++++++++++++++++++++++++++++++++---
1 file changed, 69 insertions(+), 4 deletions(-)
diff --git a/libavutil/riscv/intmath.h b/libavutil/riscv/intmath.h
index ae9ee7775b..1f0afbc81d 100644
--- a/libavutil/riscv/intmath.h
+++ b/libavutil/riscv/intmath.h
@@ -1,4 +1,6 @@
/*
+ * Copyright © 2022-2024 Rémi Denis-Courmont.
+ *
* This file is part of FFmpeg.
*
* FFmpeg is free software; you can redistribute it and/or
@@ -23,6 +25,7 @@
#include "config.h"
#include "libavutil/attributes.h"
+#include "libavutil/riscv/cpu.h"
/*
* The compiler is forced to sign-extend the result anyhow, so it is faster to
@@ -70,12 +73,74 @@ static av_always_inline av_const int av_clip_intp2_rvi(int a, int p)
}
#if defined (__GNUC__) || defined (__clang__)
-#define av_popcount __builtin_popcount
-#if (__riscv_xlen >= 64)
-#define av_popcount64 __builtin_popcountl
+static inline av_const int av_popcount_rv(unsigned int x)
+{
+#if HAVE_RV && !defined(__riscv_zbb)
+ if (!__builtin_constant_p(x) &&
+ __builtin_expect(ff_rv_zbb_support(), true)) {
+ int y;
+
+ __asm__ (
+ ".option push\n"
+ ".option arch, +zbb\n"
+#if __riscv_xlen >= 64
+ "cpopw %0, %1\n"
#else
-#define av_popcount64 __builtin_popcountll
+ "cpop %0, %1\n"
+#endif
+ ".option pop" : "=r" (y) : "r" (x));
+ if (y > 32)
+ __builtin_unreachable();
+ return y;
+ }
+#endif
+ return __builtin_popcount(x);
+}
+#define av_popcount av_popcount_rv
+
+static inline av_const int av_popcount64_rv(uint64_t x)
+{
+#if HAVE_RV && !defined(__riscv_zbb) && __riscv_xlen >= 64
+ if (!__builtin_constant_p(x) &&
+ __builtin_expect(ff_rv_zbb_support(), true)) {
+ int y;
+
+ __asm__ (
+ ".option push\n"
+ ".option arch, +zbb\n"
+ "cpop %0, %1\n"
+ ".option pop" : "=r" (y) : "r" (x));
+ if (y > 64)
+ __builtin_unreachable();
+ return y;
+ }
#endif
+ return __builtin_popcountl(x);
+}
+#define av_popcount64 av_popcount64_rv
+
+static inline av_const int av_parity_rv(unsigned int x)
+{
+#if HAVE_RV && !defined(__riscv_zbb)
+ if (!__builtin_constant_p(x) &&
+ __builtin_expect(ff_rv_zbb_support(), true)) {
+ int y;
+
+ __asm__ (
+ ".option push\n"
+ ".option arch, +zbb\n"
+#if __riscv_xlen >= 64
+ "cpopw %0, %1\n"
+#else
+ "cpop %0, %1\n"
+#endif
+ ".option pop" : "=r" (y) : "r" (x));
+ return y & 1;
+ }
+#endif
+ return __builtin_parity(x);
+}
+#define av_parity av_parity_rv
#endif
#endif /* AVUTIL_RISCV_INTMATH_H */
--
2.45.1
_______________________________________________
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 prev parent reply other threads:[~2024-06-08 11:37 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-06-08 11:37 [FFmpeg-devel] [PATCH 1/4] riscv: probe for Zbb extension at load time Rémi Denis-Courmont
2024-06-08 11:37 ` [FFmpeg-devel] [PATCH 2/4] lavu/riscv: use Zbb REV8 at run-time Rémi Denis-Courmont
2024-06-08 11:37 ` Rémi Denis-Courmont [this message]
2024-06-08 11:37 ` [FFmpeg-devel] [PATCH 4/4] lavu/riscv: use Zbb CLZ/CTZ/CLZW/CTZW " Rémi Denis-Courmont
2024-06-08 18:01 ` Lynne via ffmpeg-devel
2024-06-08 18:17 ` Rémi Denis-Courmont
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=20240608113717.1677043-3-remi@remlab.net \
--to=remi@remlab.net \
--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