-
-
Notifications
You must be signed in to change notification settings - Fork 11.2k
BLD: Prevent -flto from optimising long double representation check away #14827
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
Conversation
I don't think the failure is related to this change? |
The failing test does look unrelated, the 10 warnings are:
EDIT: Seems to come during the Cython installation. We should not be counting those warnings. |
See #14197 (comment) about the warnings. |
LGTM, but needs a comment explaining the odd construction. |
Maybe add a commented |
Thanks @chrisburr . I assume you want this backported to 1.17. |
This was required for Python 3.8 builds for macOS on conda-forge: conda-forge/numpy-feedstock#175
If I understand correctly,
-flto
with clang 9 was optimisng the teststruct
in thelong_double_representation
detection away causing the binary parsing to fail. This patch was enough to prevent this.A failing log can be found here. I think Azure will clean it at some point so I'll copy the interesting part:
The compiled binary was:
Click me to expand