KEMBAR78
Add Apple Silicon support when building wheels by 97littleleaf11 · Pull Request #10145 · python/mypy · GitHub
Skip to content

Conversation

97littleleaf11
Copy link
Collaborator

@97littleleaf11 97littleleaf11 commented Feb 26, 2021

Description

related to #10114

I add a CIBW_TEST_SKIP in line 53 according to this. Is it necessary? I am not sure whether it would work or not.

@henryiii
Copy link
Contributor

You should be correct, the explicit skip acknowledges the fact you know you can’t test arm64 wheels on Intel.

Copy link
Collaborator

@JukkaL JukkaL left a comment

Choose a reason for hiding this comment

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

Great! Let's try this out. If the build works, I'm going to try using the wheel on my Apple Silicon mac.

@JukkaL JukkaL merged commit 5161d93 into python:master Feb 27, 2021
@henryiii
Copy link
Contributor

I can test too if there's a place to download it from.

@JukkaL
Copy link
Collaborator

JukkaL commented Feb 27, 2021

@JukkaL
Copy link
Collaborator

JukkaL commented Feb 27, 2021

A quick benchmark was 4.3x faster when compiled on Apple Silicon. Nice!

@97littleleaf11 97littleleaf11 deleted the modify-build-wheel branch February 27, 2021 17:43
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.

3 participants