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 unconditionally adds -fno-strict-aliasing #74

Closed
mathstuf opened this issue Jul 6, 2017 · 2 comments
Closed

CMake unconditionally adds -fno-strict-aliasing #74

mathstuf opened this issue Jul 6, 2017 · 2 comments

Comments

@mathstuf
Copy link
Contributor

mathstuf commented Jul 6, 2017

This flag only really makes sense on GCC or GCC-like compilers. Its use should be shadowed behind a compiler check. I see that an issue was raised which caused its addition, though I suspect that the core issue should be fixed rather than relying on compiler flags to not make optimizations (e.g., MSVC, Intel, etc. all are still free to make the troublesome assumptions).

@hartwork
Copy link
Member

hartwork commented Jul 6, 2017

Hello Ben,

in discussion with the Pascal — the reporter of the issue you linked to — we decided to go with -fno-strict-aliasing in general last time. I'm happy about pull requests addressing the core issue as well as pull request guarding the adding in the CMake build system. (I will not have the resources for much energy down this road myself.) We are understaffed on CMake in general so we welcome help with CMake, e.g. see #43.

Best, Sebastian

@DerDakon
Copy link
Contributor

DerDakon commented Aug 6, 2017

Fixed by #107, can be closed.

@hartwork hartwork closed this as completed Aug 6, 2017
hartwork added a commit that referenced this issue Aug 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants