fix/mlx-c-0.6.0-metal-toolchain-compat #3

Merged

peterswimm merged 2 commits from fix/mlx-c-0.6.0-metal-toolchain-compat into main

2026-08-12 23:09:43 +00:00

peterswimm commented

2026-08-12 21:34:37 +00:00

Owner

No description provided.

peterswimm added 2 commits

2026-08-12 21:34:37 +00:00

fix(quantization): bump mlx-c submodule to v0.6.0, fix resulting ABI breaks 5ce3fd8ad0

mlx-c 0.6.0 pins mlx v0.31.1 (up from mlx-c 0.5.0's v0.30.6) — a coherent,
upstream-tested pairing. mlx_quantize/mlx_dequantize/mlx_qqmm each gained a
new required `global_scale` mlx_array parameter in this bump; pass
mlx_array_new() (the same "not provided" sentinel already used for the
optional `biases` param in this file) to keep behavior identical to before
the bump — none of these functions expose a global_scale concept in their
public Rust API today.

Does not touch anything Metal-shader-related. Evaluated as a possible fix
for a Metal-toolchain compile error some downstream consumers are hitting
on newer Xcode (MMATile/frag_at template + address-space errors in
mlx/backend/metal/kernels/steel/gemm/mma.h) — confirmed via a real build
that it does not fix that issue, and separately confirmed the same is true
at mlx v0.32.0 (an unsupported pairing, not part of this change). That
Metal issue is upstream in mlx's own shader code, unresolved in any
release through v0.32.0, and out of scope here — this PR is purely the
mlx-c version-sync + resulting compat fix, valuable independent of that
issue (e.g. for non-macOS-Metal consumers of this crate).

fix(mlx-c): point submodule at Toilville fork carrying the Metal 4.1 compat fix

validate / checks (pull_request) Failing after 3s

Details

validate / tests (macos-15, 1.85.0) (pull_request) Failing after 5s

Details

validate / tests (macos-15, stable) (pull_request) Failing after 5s

Details

072ac2eb04

mlx-c's own build (the CMakeLists.txt bump to mlx main commit
ef5fc0fab27c + the resulting fft.cpp FFTNorm patch) needs a place to
live that isn't upstream ml-explore/mlx-c -- these are internal,
not-upstreamable patches. Forked to git.toilville.dev/Toilville/mlx-c,
matching how this repo itself already forks mlx-rs/mlx-lm for the same
reason.

See ~/.spel/models/mlx-local-runner.spel for the bisection evidence.

peterswimm merged commit 2a293f4371 into main

2026-08-12 23:09:43 +00:00

peterswimm deleted branch fix/mlx-c-0.6.0-metal-toolchain-compat

2026-08-12 23:09:43 +00:00

peterswimm referenced this pull request from a commit

2026-08-12 23:09:45 +00:00

Merge pull request 'fix/mlx-c-0.6.0-metal-toolchain-compat' (#3) from fix/mlx-c-0.6.0-metal-toolchain-compat into main

peterswimm referenced this pull request

2026-08-26 23:59:12 +00:00

fix/mlx-c-0.6.0-metal-toolchain-compat: mlx-c submodule URL is malformed (missing ssh://) #4