Skip to content

ci: add job for macos-26 arm64#135

Open
jorgsowa wants to merge 1 commit into
dstogov:masterfrom
jorgsowa:ci/add-macos-aarch64
Open

ci: add job for macos-26 arm64#135
jorgsowa wants to merge 1 commit into
dstogov:masterfrom
jorgsowa:ci/add-macos-aarch64

Conversation

@jorgsowa

@jorgsowa jorgsowa commented Apr 5, 2026

Copy link
Copy Markdown

No description provided.

@jorgsowa jorgsowa changed the title ci: add job for macos-15 arm64 ci: add job for macos-26 arm64 Apr 5, 2026
@dstogov

dstogov commented Apr 6, 2026

Copy link
Copy Markdown
Owner

Unfortunately, MacOS/arm64 is not ready to be enabled. At least, there are some inconsistencies with disassembler that leads to false positive failures.

In e4451c4 I've added the test case from issue #134.
You may rebase this PR and re-run the tests.
I don't know how it may fail and how this may be related to MacOS.
This optimization is system independent.

@jorgsowa jorgsowa force-pushed the ci/add-macos-aarch64 branch from 62d4ffe to a957ee8 Compare April 6, 2026 14:51
@dstogov

dstogov commented Apr 6, 2026

Copy link
Copy Markdown
Owner

You can see https://github.com/dstogov/ir/actions/runs/24036658133/job/70100190123?pr=135 ./tests/cond_opt_001.irt is passed.

@jorgsowa

jorgsowa commented Apr 7, 2026

Copy link
Copy Markdown
Author

Yes, that's true. I run it again and works fine. Maybe it was old codebase, or I messed up with something, sorry.

Should I keep open this PR to add ARM macos to CI pipeline?

@dstogov

dstogov commented Apr 8, 2026

Copy link
Copy Markdown
Owner

Should I keep open this PR to add ARM macos to CI pipeline?

Yeah, keep this open.
I won't merge this before fixing all the tests failures.
It seems most of them are false positive mismatches in disassembler output, but few are the real bugs.
However, mac support is not in the top of my TODO list.

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