-
Notifications
You must be signed in to change notification settings - Fork 404
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
Win32 Debug Performance #526
Comments
If the only reason to switch to c++17 is to get alignment working under win32, I have a different alignment solution, I have also solved the priority inversion problem under windows 32 & 64 using a private heap with HEAP_NO_SERIALIZE. |
Well the primary reason to switch was to allow the alignas() keyword to work, so yeah that is what motivated it. I'd like to imagine there's a day where interesting sounding changes like the one you outline could get into the code base and into the hands of the users who download the synth from here. Hope we can figure out what's going wrong. |
My alignment solution is handled by my code so it cannot break regardless of the compiler. The code to fix the priority inversion you will not like and the most work on it requires removing std structures like ""list>"" and it is currently platform dependent (win only). push.back apparently calls new/malloc so I had to replace it. Luckily there is already "max_voices". |
Ahh yeah ... well that does sound a bit less mergeable! :) But still I would like to solve this problem so you can track mainline (bidirectionally) as much as possible. Many of your changes are ones which would be great to integrate I'm sure. |
@sagantech reports that running the debugger on current codebase is prohibitively slow 32 bit debug win 7 build. Here's the facts we know so far
The text was updated successfully, but these errors were encountered: