Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

arm_cfft_q15 slower than deprecated arm_cfft_radix2_q15 #467

Closed
cchuravy opened this issue Nov 20, 2018 · 2 comments
Closed

arm_cfft_q15 slower than deprecated arm_cfft_radix2_q15 #467

cchuravy opened this issue Nov 20, 2018 · 2 comments
Assignees
Labels
DSP enhancement moved moved to new github repository

Comments

@cchuravy
Copy link

I am migrating a piece of signal processing code from one M0 based MCU to another, and noticed that the arm_cfft_radix2_q15 function has been deprecated in favor of the more general purpose arm_cfft_q15 with the promise of being faster.

However, bench marking between the two files, I discovered that the new function takes >3x as long as the older one. I'm running on an STM32F091RC at 48MHz. arm_cfft_q15 takes approximately 17ms, where arm_cfft_radix2_q15 takes approximately 5ms to yield the same result.

Takes ~17ms
q15_t input_buffer[1024];
arm_cfft_radix2_instance_q15 S;
arm_cfft_radix2_init_q15(&S, 512, 0, 1);
arm_cfft_radix2_q15(&S, input_buffer);

Takes ~5ms
q15_t input_buffer[1024];
arm_cfft_q15(&arm_cfft_sR_q15_len512, input_buffer, 0, 1);

@llefaucheur
Copy link
Contributor

Hi cchuravy. Thank you for this valid comment. We reproduce it : the radix4 implementation is slower than the radix2 one for M0, which is not the case for all processors except M0/M3. We take your point for the next revision of the library. Regards, Laurent.

@christophe0606
Copy link
Contributor

We are moving CMSIS-DSP to a new repository : https://github.com/ARM-software/CMSIS-DSP

I have recreated the github issues on the new repository (sometimes merging a few related ones). You can see the link just above this comment (... mentioned this issue ...).

So, I am closing the github issue here.

@christophe0606 christophe0606 added the moved moved to new github repository label Jun 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DSP enhancement moved moved to new github repository
Projects
None yet
Development

No branches or pull requests

4 participants