[FFmpeg-cvslog] x86/tx_float: correctly load the transform length
Lynne
git at videolan.org
Sun Jul 18 16:05:17 EEST 2021
ffmpeg | branch: master | Lynne <dev at lynne.ee> | Sun Jul 18 15:00:48 2021 +0200| [997f9bdb99de18b27f8f213f381c5e90536a1be9] | committer: Lynne
x86/tx_float: correctly load the transform length
The field is a standard field, yet we were loading it as if it was
a quadword. This worked for forward transforms by chance, but broke
when the transform was inverse.
checkasm couldn't catch that because we only test forward transforms,
which are identical to inverse transforms but with a different revtab.
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=997f9bdb99de18b27f8f213f381c5e90536a1be9
---
libavutil/x86/tx_float.asm | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavutil/x86/tx_float.asm b/libavutil/x86/tx_float.asm
index b71fb49592..4d2283fae1 100644
--- a/libavutil/x86/tx_float.asm
+++ b/libavutil/x86/tx_float.asm
@@ -884,7 +884,7 @@ ALIGN 16
%macro FFT_SPLIT_RADIX_FN 1
INIT_YMM %1
cglobal split_radix_fft_float, 4, 8, 16, 272, lut, out, in, len, tmp, itab, rtab, tgt
- mov lenq, [lutq + AVTXContext.m]
+ movsxd lenq, dword [lutq + AVTXContext.m]
mov lutq, [lutq + AVTXContext.revtab]
mov tgtq, lenq
More information about the ffmpeg-cvslog
mailing list