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

add apple chips to arm CPU detection #17837

Merged
merged 1 commit into from
Nov 3, 2023
Merged

Conversation

xdBronch
Copy link
Contributor

@xdBronch xdBronch commented Nov 3, 2023

before this change if someone were to run apple silicon on e.g. Asahi linux zig would never search for apple chips and default to a generic cpu. the part numbers for the M1 and implementer number were found by reading /proc/cpuinfo on an M1 running asahi linux, the numbers for the M2 i found here which i cannot confirm but the numbers fall in line with the M1 so i think its safe to trust.
i can do a follow up for the M3 if/when I can find the info I need and it's added to zigs list of targets

@kubkon
Copy link
Member

kubkon commented Nov 3, 2023

It would be nice to properly handle page size when running Linux on Apple Silicon - I don't think we handle this properly atm.

@xdBronch
Copy link
Contributor Author

xdBronch commented Nov 3, 2023

definitely yeah, I believe there's a PR open right now for that so it should hopefully be solved

@andrewrk andrewrk changed the title dont assume apple chips are macos exclusive add apple chips to arm CPU detection Nov 3, 2023
@andrewrk
Copy link
Member

andrewrk commented Nov 3, 2023

Thanks!

@andrewrk andrewrk merged commit 5bd27a2 into ziglang:master Nov 3, 2023
10 checks passed
@xdBronch xdBronch deleted the apple-linux branch November 3, 2023 19:19
@kamidev
Copy link

kamidev commented Nov 3, 2023

definitely yeah, I believe there's a PR open right now for that so it should hopefully be solved

@kubkon this is the PR: #17382, if you are interested. It seemed close, but may need someone to guide it through CI.

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.

4 participants