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

Thunks: Removes global static initializer #4021

Merged

Conversation

Sonicadvance1
Copy link
Member

This variable can't be constexpr initialized since it requires linker fix-ups, which changes it in to a global static initializer instead.

While this benign as it doesn't allocate any memory, just move it next to its single use. Removes the static initializer that was there for no reason.

This variable can't be constexpr initialized since it requires linker
fix-ups, which changes it in to a global static initializer instead.

While this benign as it doesn't allocate any memory, just move it next
to its single use. Removes the static initializer that was there for no
reason.
@Sonicadvance1 Sonicadvance1 merged commit b4a67a6 into FEX-Emu:main Sep 1, 2024
12 checks passed
@Sonicadvance1 Sonicadvance1 deleted the remove_global_initializer_thunks branch September 1, 2024 05:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants