Skip to content

ggml-impl.h: Fix build issues on AArch64 with CUDA version 12 #12872

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

Closed

Conversation

zhengjun-xing
Copy link

Fix #12732:

  • Remove incorrect inclusion of "arm_neon.h" for CUDA versions ≥ 12

Make sure to read the contributing guidelines before submitting a PR

Fix ggml-org#12732:
* Remove incorrect inclusion of "arm_neon.h" for CUDA versions ≥ 12
@zhengjun-xing zhengjun-xing force-pushed the fix-build-aarch64-cuda12 branch from 725be9c to d85170b Compare April 10, 2025 10:29
@github-actions github-actions bot added the ggml changes relating to the ggml tensor library for machine learning label Apr 10, 2025
@ggerganov
Copy link
Member

Should be fixed with cb79c2e

@ggerganov ggerganov closed this Apr 10, 2025
@zhengjun-xing
Copy link
Author

Thanks for merging this PR! Just curious—why was the author name changed? Is that typical in open source workflows?

@zhengjun-xing
Copy link
Author

zhengjun-xing commented Apr 11, 2025 via email

@zhengjun-xing
Copy link
Author

Oops, I initially thought the PR was merged, but it looks like it was actually closed without being merged.

@ggerganov
Copy link
Member

Oops, I initially thought the PR was merged, but it looks like it was actually closed without being merged.

The same fix was proposed in parallel so we ended up merging the other PR:

ggml-org/ggml#1187

The final result should be the same. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ggml changes relating to the ggml tensor library for machine learning
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants