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

Release optimization options #37

Open
antoniovazquezblanco opened this issue Apr 4, 2023 · 1 comment
Open

Release optimization options #37

antoniovazquezblanco opened this issue Apr 4, 2023 · 1 comment

Comments

@antoniovazquezblanco
Copy link
Contributor

Currently -O3 -ffast-math -s for release builds. Do we want to throw it ot the window and use -Ofast?

https://gcc.gnu.org/onlinedocs/gcc/Optimize-Options.html
@antoniovazquezblanco antoniovazquezblanco mentioned this issue Apr 5, 2023
Merged
@BatchDrake
Copy link
Owner

I completely forgot about -ffast-math (maybe because I always build in debug mode), and now I am worried about it. While nobody actually complained, -ffast-math is a delicate subject in the sense that reorders math instructions in a way that is not completely IEEE-compliant. This may affect stuff like Kahan summations, in which the order is important.

I am leaving this issue open so I do not forget to look into it. But some thorough testing will be necessary here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@antoniovazquezblanco @BatchDrake