search

ARM-software / CMSIS-DSP

CMSIS-DSP embedded compute library for Cortex-M and Cortex-A
https://arm-software.github.io/CMSIS-DSP
Apache License 2.0
518 stars 133 forks source link

Matrix Vector Multiplication fails for large matrix size #118

Closed yuvpg closed 8 months ago

yuvpg commented 1 year ago

arm_mat_vec_mult_f32() fails to produce correct output if rowCount*colCount > 65536

There is a bug in definition of a loop counter:

uint16_t i, row, colCnt; /* loop counters */    <-- should be uint32_t

https://github.com/ARM-software/CMSIS-DSP/blob/cb0960577d6483a2a626ae5d8778b6f00a844b32/Source/MatrixFunctions/arm_mat_vec_mult_f32.c#L301C5-L301C14

christophe0606 commented 1 year ago

@yuvpg Thanks for reporting this.

christophe0606 commented 10 months ago

It should be fixed in latest commit.