]> git.djapps.eu Git - pkg/ggml/sources/llama.cpp/commit
vulkan: fail the build when a shader fails to compile (#24450)
authorliminfei-amd <redacted>
Wed, 24 Jun 2026 09:42:03 +0000 (17:42 +0800)
committerGitHub <redacted>
Wed, 24 Jun 2026 09:42:03 +0000 (11:42 +0200)
commit1191758c5d77e575b4531e057708a562f6f6f0b2
treea467a3b63a6ff12bb67d50f2be57e74f962bf4c3
parent00139b660b3a61e130aa76dcc7cb9e447da43898
vulkan: fail the build when a shader fails to compile (#24450)

* vulkan-shaders-gen: fail the build when a shader fails to compile

vulkan-shaders-gen did not detect shader-compile subprocess failures, so a
broken libggml-vulkan could be produced while the build reported success and
the breakage only surfaced at run time. execute_command() discarded the child
exit code (POSIX waitpid passed nullptr for status; the Windows branch never
called GetExitCodeProcess) and string_to_spv decided success only from whether
stderr was empty, so a non-zero exit with empty stderr, or a subprocess that
failed to launch, was treated as success.

Return the child exit code from execute_command() (WEXITSTATUS on POSIX,
GetExitCodeProcess on Windows), treat a non-zero exit or non-empty stderr or a
launch exception as a failure, and record it in an atomic flag. main() checks
the flag after process_shaders() and returns EXIT_FAILURE before writing the
output files, so the build stops instead of emitting a broken backend.

Fixes #24393

Signed-off-by: liminfei-amd <redacted>
* vulkan-shaders-gen: simplify compile_failed access and drop unreachable return

Address review feedback on #24450:
- Access the std::atomic<bool> compile_failed directly (= / implicit bool)
  instead of .store()/.load(); the flag stays atomic because the worker
  threads in process_shaders() set it concurrently.
- Remove the unreachable trailing return -1 in execute_command(): on POSIX the
  child _exit()s after execvp and the parent returns (fork()<0 throws); on
  Windows the block returns the exit code.

Signed-off-by: liminfei-amd <redacted>
---------

Signed-off-by: liminfei-amd <redacted>
ggml/src/ggml-vulkan/vulkan-shaders/vulkan-shaders-gen.cpp