[FFmpeg-devel] [PATCH 2/2] avutil/float_dsp: add ff_vector_dmul_{sse2, avx}

Henrik Gramner henrik at gramner.com
Fri Sep 14 15:57:10 EEST 2018


On Thu, Sep 13, 2018 at 3:08 PM, James Almer <jamrial at gmail.com> wrote:
> +    lea       lenq, [lend*8 - mmsize*4]

Is len guaranteed to be a multiple of mmsize/8? Otherwise this would
cause misalignment. It will also break if len < mmsize/2.

Also if you want a 32-bit result from lea it should be written as "lea
lend, [lenq*8 - mmsize*4]" which is equivalent but has a shorter
opcode (e.g. always use native sizes within brackets).


More information about the ffmpeg-devel mailing list