Skip to content

fix compilation with MSVC for arm64 with __ARM_NEON defined #3233

Closed
divideconcept wants to merge 2 commits intoggml-org:masterfrom
divideconcept:master
Closed

fix compilation with MSVC for arm64 with __ARM_NEON defined #3233
divideconcept wants to merge 2 commits intoggml-org:masterfrom
divideconcept:master

Conversation

@divideconcept
Copy link
Copy Markdown

These 2 commits fix compilation errors when trying to build with MSVC arm64 with __ARM_NEON defined

__fp16 is an undeclared identifie with MSVC
fix vaddlvq_s16 already defined as _CopyInt32FromFloat by MSVC arm64, and other similar functions
Comment thread ggml/src/ggml-impl.h
Comment thread ggml/src/ggml-cpu/ggml-cpu-impl.h
@ggerganov
Copy link
Copy Markdown
Member

This is a bit complicated to explain, but the ggml generally does not support Arm + MSVC. The recommended alternative is Arm + Clang. In short, this fix won't work.

See ggml-org/ggml#1177 and references there.

@ggerganov ggerganov closed this Jun 9, 2025
@divideconcept
Copy link
Copy Markdown
Author

FWIW, with this patch it builds and it's properly running, doing speech to text as expected. I don't know if it actually uses NEON instructions though.

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.

3 participants