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

metal : another fix for the fa kernel #9188

Closed
wants to merge 1 commit into from
Closed

Conversation

ggerganov
Copy link
Owner

@ggerganov ggerganov commented Aug 26, 2024

alt #9187

Technically, this check is redundant AFAIU, but somehow it fixes the kernel on M1 Pro and M2 Ultra

@ggerganov ggerganov mentioned this pull request Aug 26, 2024
4 tasks
@slaren
Copy link
Collaborator

slaren commented Aug 26, 2024

This also works on M3 Max.

@ggerganov
Copy link
Owner Author

Superseded by #9189

@ggerganov ggerganov closed this Aug 26, 2024
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