Skip to content

Aarch64 darwin #16

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ricardo-valero
Copy link

Just a humble peasant submitting a PR. 😅

I added support for macOS Silicon. I tested it with the counter and lexer examples, and everything seems to be working fine.

I borrowed some snippets from other PRs but since they haven't been merged yet, I wasn't sure what the best approach was—so I just went for it.

I had to comment out these lines, because I get this
error: non-ASM statement in naked function is not supported
@Eropi4
Copy link

Eropi4 commented Feb 9, 2025

I can confirm it works on M2!

@nevakrien
Copy link

there has to be a better way to organize arches... like if you look in musl the way they do long jump is that every arch has a small file with just the calling convention nonsense in it.
that would make merging pull request for multiple arches easy so probably a big win

@nikpivkin
Copy link

Used this version for koil and works pretty good! However, sometimes the server crashes when players exit, but I don't know if it's related to this change.

RROR: could not read message from player 2
ERROR: could not read message from player 3
./build/server [] exited with null

@davttd
Copy link

davttd commented May 4, 2025

This is amazing thanks for providing this!

@andrei-golubov
Copy link

Tested on M1, looks good!

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.

6 participants