Skip to content

Add Apple M5/M5 Pro/M5 Max memory bandwidth entries#551

Merged
AlexsJones merged 2 commits into
AlexsJones:mainfrom
denisdrake:add-m5-bandwidth
May 19, 2026
Merged

Add Apple M5/M5 Pro/M5 Max memory bandwidth entries#551
AlexsJones merged 2 commits into
AlexsJones:mainfrom
denisdrake:add-m5-bandwidth

Conversation

@denisdrake
Copy link
Copy Markdown
Contributor

Summary

  • Adds M5 (153.6 GB/s), M5 Pro (307 GB/s), and M5 Max (614 GB/s) to the Apple Silicon GPU memory bandwidth lookup table
  • Includes corresponding test assertions

Motivation

The M5 Max was released March 2026 and is already shipping in MacBook Pro 14"/16". Without these entries, llmfit falls back to a generic TPS estimate for M5 hardware, producing inaccurate speed predictions. The M5 Max at 614 GB/s is ~12.5% faster than M4 Max (546 GB/s).

Test plan

  • cargo test -p llmfit-core test_gpu_bandwidth_apple_silicon passes
  • All bandwidth tests pass (test_gpu_bandwidth_known_gpus, test_gpu_bandwidth_amd, test_gpu_bandwidth_unknown_returns_none)

Sources: Apple tech specs, notebookcheck.net

🤖 Generated with Claude Code

M5 Max (40-core): 614 GB/s, M5 Pro: 307 GB/s, base M5: 153.6 GB/s.
These are the 2025/2026 Apple Silicon models missing from the lookup table.

Sources: Apple tech specs, notebookcheck.net benchmarks.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@AlexsJones
Copy link
Copy Markdown
Owner

thanks for this - please cargo fmt!

@denisdrake
Copy link
Copy Markdown
Contributor Author

👋 Friendly ping — this is a tiny 3-line change, tested. M5 Max is already shipping in the wild (I'm running llmfit on one right now). Would be great to get this in!

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@denisdrake
Copy link
Copy Markdown
Contributor Author

Done — cargo fmt applied and pushed. Should be clean now.

@AlexsJones AlexsJones merged commit 1aa22bc into AlexsJones:main May 19, 2026
8 checks passed
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