summaryrefslogtreecommitdiff
path: root/libavutil/tx.c
diff options
context:
space:
mode:
authorLynne <dev@lynne.ee>2022-09-03 03:36:40 +0200
committerLynne <dev@lynne.ee>2022-09-06 04:21:46 +0200
commit4537d9554d9a0027ba028c8eb16221d9235ebe43 (patch)
tree1924f67b3164f800d23cb907946050463506f2df /libavutil/tx.c
parent2425d5cd7e37387305f85bef63f7441c8b1cc147 (diff)
downloadffmpeg-4537d9554d9a0027ba028c8eb16221d9235ebe43.tar.gz
x86/tx_float: implement inverse MDCT AVX2 assembly
This commit implements an iMDCT in pure assembly. This is capable of processing any mod-8 transforms, rather than just power of two, but since power of two is all we have assembly for currently, that's what's supported. It would really benefit if we could somehow use the C code to decide which function to jump into, but exposing function labels from assebly into C is anything but easy. The post-transform loop could probably be improved. This was somewhat annoying to write, as we must support arbitrary strides during runtime. There's a fast branch for stride == 4 bytes and a slower one which uses vgatherdps. Zen 3 benchmarks for stride == 4 for old (av_imdct_half) vs new (av_tx): 128pt: 2811 decicycles in av_tx (imdct),16775916 runs, 1300 skips 3082 decicycles in av_imdct_half,16776751 runs, 465 skips 256pt: 4920 decicycles in av_tx (imdct),16775820 runs, 1396 skips 5378 decicycles in av_imdct_half,16776411 runs, 805 skips 512pt: 9668 decicycles in av_tx (imdct),16775774 runs, 1442 skips 10626 decicycles in av_imdct_half,16775647 runs, 1569 skips 1024pt: 19812 decicycles in av_tx (imdct),16777144 runs, 72 skips 23036 decicycles in av_imdct_half,16777167 runs, 49 skips
Diffstat (limited to 'libavutil/tx.c')
-rw-r--r--libavutil/tx.c19
1 files changed, 10 insertions, 9 deletions
diff --git a/libavutil/tx.c b/libavutil/tx.c
index edd2af07c7..da8ebddd9a 100644
--- a/libavutil/tx.c
+++ b/libavutil/tx.c
@@ -206,23 +206,24 @@ static void parity_revtab_generator(int *revtab, int n, int inv, int offset,
1, 1, len >> 1, basis, dual_stride, inv_lookup);
}
-int ff_tx_gen_split_radix_parity_revtab(AVTXContext *s, int invert_lookup,
- int basis, int dual_stride)
+int ff_tx_gen_split_radix_parity_revtab(AVTXContext *s, int len, int inv,
+ int inv_lookup, int basis, int dual_stride)
{
- int len = s->len;
- int inv = s->inv;
-
- if (!(s->map = av_mallocz(len*sizeof(*s->map))))
- return AVERROR(ENOMEM);
-
basis >>= 1;
if (len < basis)
return AVERROR(EINVAL);
+ if (!(s->map = av_mallocz((inv_lookup == -1 ? 2 : 1)*len*sizeof(*s->map))))
+ return AVERROR(ENOMEM);
+
av_assert0(!dual_stride || !(dual_stride & (dual_stride - 1)));
av_assert0(dual_stride <= basis);
+
parity_revtab_generator(s->map, len, inv, 0, 0, 0, len,
- basis, dual_stride, invert_lookup);
+ basis, dual_stride, inv_lookup != 0);
+ if (inv_lookup == -1)
+ parity_revtab_generator(s->map + len, len, inv, 0, 0, 0, len,
+ basis, dual_stride, 0);
return 0;
}