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.

Sign up for GitHub

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

Jump to bottom

SIMD status for these plugins? #31

Open
jamshark70 opened this issue Feb 5, 2022 · 2 comments
Open

SIMD status for these plugins? #31

jamshark70 opened this issue Feb 5, 2022 · 2 comments
Labels
help wanted Extra attention is needed

Comments

@jamshark70
Copy link

I'm curious if these plugins are getting SIMD-optimized during compilation. I suspect not, as they are very CPU hungry (e.g. 2 instances of AnalogTape = 17-18% CPU). So, for instance, I might use the VA filters for a monophonic bass line but I would have to avoid too many of them at once.

I'm afraid my C++ skills are not good enough to review the plugins for this. Spacechild1 had pointed out to me on the SC forum that 8-sample loops can clue the compiler in to places where SIMD instructions could be used. This resulted in a massive speedup for me.
@madskjeldgaard
Copy link
Owner

I agree - I think this would be awesome. I don't think (this is the disadvantage of only porting other's code and not writing it yourself from the ground up) that it's not simd optimized.

If anyone has pointers on how to do it, I can look into it and mess around with it at some point. As an alternative we could also port the Surge ports of the same, they seem to have someting along those lines going on?

https://github.com/surge-synthesizer/surge/blob/main/src/common/dsp/filters/K35Filter.cpp#L111

https://github.com/surge-synthesizer/surge/blob/0b36ab24d0fe2a1810d29f57e0e0a0cee8af0013/doc/Adding%20A%20Filter.md

@madskjeldgaard
Copy link
Owner

It would be really awesome to fix all three of these filter related issues(#30 and #28). They're all big and important improvements IMO

@madskjeldgaard madskjeldgaard added the help wanted Extra attention is needed label Feb 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jamshark70 @madskjeldgaard