Skip to content

vulkan: split ggml-vulkan.cpp file - #24905

Draft
0cc4m wants to merge 1 commit into
masterfrom
0cc4m/vulkan-cpp-split
Draft

0cc4m wants to merge 1 commit into
masterfrom
0cc4m/vulkan-cpp-split

Conversation

@0cc4m

@0cc4m 0cc4m commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Overview

This is the first step to split up the ggml-vulkan.cpp file into multiple files. I have a Python script that hopefully allows keeping this PR up to date without much work while waiting for a good moment to merge it. Some of the long-running draft PRs might have to be updated with the help of the script, I'm still looking into how best to enable that.

I also still have to go through the files and see what can/should be improved. Do add feedback if you have any. The script is here.

Requirements

  • I have read and agree with the contributing guidelines
  • AI usage disclosure: YES, Claude was used to write the script.

@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 Jun 22, 2026
@Raman-Raje

Raman-Raje commented Aug 7, 2026 •

Copy link
Copy Markdown
Contributor

@0cc4m The looks neat and clean. Have one small follow-up on this. Pulling out vk_per_logger into its own ggml-vulkan-perf.cpp. Tried this in branch https://github.com/Raman-Raje/llama.cpp/tree/vulkan-cpp-split

This branch has not been deployed

No deployments
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