Open ayavilevich opened 1 year ago
Hi @ayavilevich ,
In float version of FFT we have an table base implementation of bit reverse, and that's why it's faster. We will think to add the same functionality for the int16 version.
About dsps_cplx2real_sc16 the approach is different. The ansi version and asm version will not have such big difference as FFT or other functions, that's why we have onlu ansi version for this function.
Regards, Dmitry
It looks like there is just an ansi implementation of
dsps_bit_rev_sc16
at this time. On the other hand, there is an optimized implementation of thedsps_bit_rev2r_fc32
but the fc32 FFT is not accelerated where the sc16 one is.Considering the optimized
dsps_fft2r_sc16
on the S3, it is a blocker for any effective, fast and fixed point FFT on the S3 to not have an optimized bit reversal function as well. Currently, if the first step of the FFT takes time T then just the final bit reversal (ansi) will take 2T.Please consider adding. I am not sure about the effort, but maybe it is similar to the floating point version with just type changes?!
P.S. similar issue with the dsps_cplx2real_sc16, but lower priority.