[FFmpeg-cvslog] avcodec/xbmdec: Use LUT for hex char->number conversion
Andreas Rheinhardt
git at videolan.org
Sun Mar 10 14:58:58 EET 2024
ffmpeg | branch: master | Andreas Rheinhardt <andreas.rheinhardt at outlook.com> | Wed Mar 6 22:49:54 2024 +0100| [0b2ed7f6c7e44c9dbf6b79aa36b8db033d192760] | committer: Andreas Rheinhardt
avcodec/xbmdec: Use LUT for hex char->number conversion
Improves performance by about 30%.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=0b2ed7f6c7e44c9dbf6b79aa36b8db033d192760
---
libavcodec/xbmdec.c | 27 ++++++++++++++++-----------
1 file changed, 16 insertions(+), 11 deletions(-)
diff --git a/libavcodec/xbmdec.c b/libavcodec/xbmdec.c
index ecfc957a3f..2d84327e02 100644
--- a/libavcodec/xbmdec.c
+++ b/libavcodec/xbmdec.c
@@ -28,17 +28,22 @@
static int get_nibble(uint8_t x)
{
- int ret = 255;
-
- if (x <= '9') {
- if (x >= '0')
- ret = x - '0';
- } else if (x >= 'a') {
- if (x <= 'f')
- ret = x - ('a' - 10);
- } else if (x >= 'A' && x <= 'F')
- ret = x - ('A' - 10);
- return ret;
+#define TIMES256(idx) \
+TIMES64(4 * (idx)) TIMES64(4 * (idx) + 1) TIMES64(4 * (idx) + 2) TIMES64(4 * (idx) + 3)
+#define TIMES64(idx) \
+TIMES16(4 * (idx)) TIMES16(4 * (idx) + 1) TIMES16(4 * (idx) + 2) TIMES16(4 * (idx) + 3)
+#define TIMES16(idx) \
+TIMES4(4 * (idx)) TIMES4(4 * (idx) + 1) TIMES4(4 * (idx) + 2) TIMES4(4 * (idx) + 3)
+#define TIMES4(idx) \
+ENTRY(4 * (idx)) ENTRY(4 * (idx) + 1) ENTRY(4 * (idx) + 2) ENTRY(4 * (idx) + 3)
+#define ENTRY(x) [x] = ((x) >= 'a' && (x) <= 'f') ? (x) - ('a' - 10) : \
+ ((x) >= 'A' && (x) <= 'F') ? (x) - ('A' - 10) : \
+ ((x) >= '0' && (x) <= '9') ? (x) - '0' : 255,
+
+ static const uint8_t lut[] = {
+ TIMES256(0)
+ };
+ return lut[x];
}
static int parse_str_int(const uint8_t *p, const uint8_t *end, const uint8_t *key)
More information about the ffmpeg-cvslog
mailing list