Skip to content

Adding support for Apple Silicon (without MPI)#428

Merged
bartgol merged 2 commits intomasterfrom
jeff-cohere/mac-mx-fpes-fix
Apr 13, 2026
Merged

Adding support for Apple Silicon (without MPI)#428
bartgol merged 2 commits intomasterfrom
jeff-cohere/mac-mx-fpes-fix

Conversation

@jeff-cohere
Copy link
Copy Markdown
Contributor

@jeff-cohere jeff-cohere commented Apr 9, 2026

This PR stubs out floating point environment logic for Apple Silicon (whose floating point environment is different and complicated and possibly something to be studied in the distant future).

Apple products are supported only without MPI, because some hwloc-related features aren't supported on Apple silicon, and this can produce failures if special steps are not taken. We can follow this up with a PR that adds support for MPI on Macs if there is interest.

Accordingly, I've also disabled the EKAT comm unit test when MPI is disabled.

Motivation

Some folks like to develop on their laptops. This allows EKAT to be built in a restricted way to support this kind of development. At the very least, mam4xx developers will be able to work on mam4xx's standalone configuration on Macs.

Testing

The PR doesn't change any testing logic, aside from disabling an MPI-related test when MPI is disabled.

@jeff-cohere jeff-cohere added the mac Related to Mac builds label Apr 9, 2026
@jeff-cohere jeff-cohere requested a review from bartgol April 9, 2026 22:14

#else

// Drop-in replacement of some useful GNU utils (needed on Apple platforms)
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It occurs to me that this logic was intended for Intel Macs, which are pretty old at this point. We could delete this code block if it's not used for anything else.

Copy link
Copy Markdown
Contributor

@bartgol bartgol left a comment

Choose a reason for hiding this comment

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

@jeff-cohere If supporting APPLE devices is important, we can add a job of the cpu-gcc workflow that runs on macos-latest...

@jeff-cohere
Copy link
Copy Markdown
Contributor Author

Thanks, Luca. I'm not sure how to add macos-latest to the existing GitHub workflow, but even without it we'll have people testing the Mac builds with mam4xx soon. Is it okay to merge, or do you think we should add Mac builds to our GitHub CI first?

@bartgol bartgol merged commit 007bdcf into master Apr 13, 2026
4 checks passed
@bartgol bartgol deleted the jeff-cohere/mac-mx-fpes-fix branch April 13, 2026 17:27
@bartgol
Copy link
Copy Markdown
Contributor

bartgol commented Apr 13, 2026

Since APPLE is not for production, it's prob ok to live without a CI. If someone is really invested in ekat to work on APPLE, they can submit a PR with the new workflow...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mac Related to Mac builds

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants