Skip to content
This repository has been archived by the owner on Jun 24, 2024. It is now read-only.

Commit

Permalink
fix(ggml): don't use Neon on macOS aarch64
Browse files Browse the repository at this point in the history
This code doesn't feel right. The original order pre-#426 should work -
why would mcpu=native not work in that case?

I think this is something that will need revisiting.
  • Loading branch information
philpax committed Nov 12, 2023
1 parent 23e4b46 commit 23c3047
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion crates/ggml/sys/build.rs
Original file line number Diff line number Diff line change
Expand Up @@ -77,9 +77,9 @@ fn main() {
if compiler.is_like_clang() || compiler.is_like_gnu() {
if target_os == "macos" {
build.flag("-mcpu=apple-m1");
build.flag("-mfpu=neon");
} else if std::env::var("HOST") == std::env::var("TARGET") {
build.flag("-mcpu=native");
build.flag("-mfpu=neon");
}
build.flag("-pthread");
}
Expand Down

0 comments on commit 23c3047

Please sign in to comment.