Skip to content

Vulkan: Clean up mul_mm shader#15987

Merged
0cc4m merged 3 commits intomasterfrom
0cc4m/vulkan-mm-cleanup
Sep 14, 2025
Merged

Vulkan: Clean up mul_mm shader#15987
0cc4m merged 3 commits intomasterfrom
0cc4m/vulkan-mm-cleanup

Conversation

@0cc4m
Copy link
Copy Markdown
Contributor

@0cc4m 0cc4m commented Sep 14, 2025

The mul_mm was the first shader I wrote and many of the readability options in GLSL weren't known to me at the time, so in this PR I move all of the dequantization stuff outside of the main file. This makes the shader much easier to work with.

I also did some minor optimizations that I spotted on the way.

It's basically a port of the good changes in #12515, while leaving out the ones that didn't work well.

@0cc4m 0cc4m requested a review from jeffbolznv September 14, 2025 12:04
@github-actions github-actions Bot added Vulkan Issues specific to the Vulkan backend ggml changes relating to the ggml tensor library for machine learning labels Sep 14, 2025
@0cc4m 0cc4m merged commit 261e6a2 into master Sep 14, 2025
45 of 48 checks passed
@0cc4m 0cc4m deleted the 0cc4m/vulkan-mm-cleanup branch September 14, 2025 14:56
blime4 referenced this pull request in blime4/llama.cpp Feb 5, 2026
* vulkan: move mul_mm dequantization steps into a separate file and functions

* improve mul_mm vector load code

* fix debug mode issues and warnings
Seunghhon pushed a commit to Seunghhon/llama.cpp that referenced this pull request Apr 26, 2026
* vulkan: move mul_mm dequantization steps into a separate file and functions

* improve mul_mm vector load code

* fix debug mode issues and warnings
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 Vulkan Issues specific to the Vulkan backend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants