-
-
Notifications
You must be signed in to change notification settings - Fork 274
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
-ffast-math
in one package might infect all libraries in an environment with inaccurate FPU
#1824
Comments
@mikemhenry mentioned on gitter (now matrix) that GCC 13 has a fix for this:
|
💯 I think a good start would be to do a search across the feedstocks to see what packages are using dangerous optimizations + the |
Could that gcc fix be a good reason to reignite efforts to package a current gcc in conda-forge on windows (#1044 I think)? I've read on the fortran lang forum that they also already felt that restriction (gcc 5!) when trying to modernise the fortran packages within python. |
That's a completely separate topic. GCC on linux (where it is our default compiler) is on 11 by default, and will be changed to 12 in the not too distant future. Fortan on windows is a different topic, where people have been waiting for flang to materialise. There's a PR to make the change to a llvm 11 based classic flang, but that's been made dependent on some infrastructure work on the migrators that no-one has tackled yet. I'm trying to build the new llvm-internal flang but even that will run into the same issue afterwards. |
A small bit of good news on this: clang is deprecating |
This blog post raises a number of concerning points, the prime one being:
This is made worse by
-Ofast
(it's gonna make my code faster, right?) turning on-ffast-math
unconditionally (as in:-fno-fast-math
has no effect).I think we should consider stripping out those flags where they exist. The blog also mentions a few ways to analyse whether the shared objects are affected. Perhaps that's something for conda-build to do after verification?
Xref: llvm/llvm-project#57589, https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55522
The text was updated successfully, but these errors were encountered: