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

cmake: never use strict aliasing #1160

Merged
merged 1 commit into from
May 29, 2024
Merged

Conversation

illwieckz
Copy link
Member

Never use strict aliasing, do the same with nexe than with exe and dll.

Alternative to:

Instead of trying to always do strict aliasing, never do strict aliasing.

What matters is to have consistent behaviour, #1159 will require more testing than this one.

See:

@slipher
Copy link
Member

slipher commented May 27, 2024

LGTM

@illwieckz illwieckz force-pushed the illwieckz/never-strict-aliasing branch from 243f40c to 69c8ced Compare May 28, 2024 09:25
@illwieckz illwieckz merged commit 37f0c6b into master May 29, 2024
9 checks passed
@illwieckz illwieckz deleted the illwieckz/never-strict-aliasing branch May 29, 2024 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants