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

MAINT: explicit disabling CCompilerOpt in F2PY #17546

Merged
merged 1 commit into from
Oct 15, 2020

Conversation

seiko2plus
Copy link
Member

Explicit disabling CCompilerOpt in F2PY

This patch also cleans up CCompilerOpt calls in build_ext and build_clib.

closes #17321

  This patch also cleans up `CCompilerOpt` calls in build_ext and build_clib.
@seiko2plus
Copy link
Member Author

@melissawm, sorry for taking 4 ever.

This patch is adding compiler defention -DNPY_DISABLE_OPTIMIZATION=1 to any involved C/C++ sources,
due to passing --disable-optimzation to f2py extention.

compile options: '-DNPY_DISABLE_OPTIMIZATION=1 -I/tmp/tmp28kqexlt/src.linux-x86_64-3.6 -I/usr/local/lib/python3.6/dist-packages/numpy-1.20.0.dev0+fe77f46-py3.6-linux-x86_64.egg/numpy/core/include -I/usr/include/python3.6m -c'
x86_64-linux-gnu-gcc: /tmp/tmp28kqexlt/src.linux-x86_64-3.6/fib1module.c
x86_64-linux-gnu-gcc: /tmp/tmp28kqexlt/src.linux-x86_64-3.6/fortranobject.c

@melissawm
Copy link
Member

I believe the CI failure is related to #17542

Otherwise, LGTM. Thanks, @seiko2plus !

@mattip mattip merged commit 99cf84a into numpy:master Oct 15, 2020
@mattip
Copy link
Member

mattip commented Oct 15, 2020

Thanks @seiko2plus

@seiko2plus seiko2plus deleted the issue_17321 branch January 9, 2021 16:53
@rgommers rgommers added the component: SIMD Issues in SIMD (fast instruction sets) code or machinery label Jul 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
03 - Maintenance component: SIMD Issues in SIMD (fast instruction sets) code or machinery
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Increased output when running f2py (possibly due to SIMD detection code)
4 participants