Skip to content

Fix building on Linux#1446

Merged
awni merged 1 commit intoml-explore:mainfrom
zcbenz:fix-linxu
Sep 30, 2024
Merged

Fix building on Linux#1446
awni merged 1 commit intoml-explore:mainfrom
zcbenz:fix-linxu

Conversation

@zcbenz
Copy link
Collaborator

@zcbenz zcbenz commented Sep 30, 2024

Proposed changes

Set MLX_BUILD_METAL only for arm64 mac, and set MLX_BUILD_ARM for arm on all platforms.

Checklist

Put an x in the boxes that apply.

  • I have read the CONTRIBUTING document
  • I have run pre-commit run --all-files to format my code / installed pre-commit prior to committing changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have updated the necessary documentation (if needed)

Copy link
Member

@awni awni left a comment

Choose a reason for hiding this comment

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

Thanks for fixing that!

@awni awni merged commit 5900e32 into ml-explore:main Sep 30, 2024
@zcbenz zcbenz deleted the fix-linxu branch October 21, 2024 23:50
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