Skip to content

Add build script for Windows on ARM64 #192

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

Merged
merged 2 commits into from
Apr 3, 2025

Conversation

Harishmcw
Copy link
Contributor

This PR adds build_steps_win_arm64.bat, a script that automates the process of building OpenBLAS and packaging it into a SciPy-compatible wheel on Windows ARM64. It compiles OpenBLAS using the ARM64 native command prompt, prepares the necessary files, and generates a wheel for seamless integration with SciPy.

  • I updated the version in pyproject.toml and made sure it matches git describe --tags --abbrev=8 in OpenBLAS at the OPENBLAS_COMMIT

Copy link
Collaborator

@mattip mattip left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't the BUILD_BITS be used to define a prefix/suffix in the exported function names? Or am I missing where this is done?

Co-authored-by: Matti Picus <matti.picus@gmail.com>
@Harishmcw
Copy link
Contributor Author

Shouldn't the BUILD_BITS be used to define a prefix/suffix in the exported function names? Or am I missing where this is done?

The suffix/prefix on exported symbols is determined based on the if_bits variable in the build_openblas.sh script, while the build_bits variable is used for configuring the OpenBLAS build.

If INTERFACE64 is set to 1, if_bits is assigned 64; otherwise, it defaults to 32. Since INTERFACE64 is not used here, if_bits remains 32, resulting in only the prefix scipy_ being added without any suffixes

@mattip
Copy link
Collaborator

mattip commented Apr 3, 2025

Ahh, I see. NumPy uses the 64-bit interfaces, so I guess more work will be needed to produce those wheels.

@mattip
Copy link
Collaborator

mattip commented Apr 3, 2025

Thanks.

@mattip mattip merged commit 29ffffb into MacPython:main Apr 3, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants