Skip to content

cuda: fix warnings for unused variable/function - #26688

Merged
ggerganov merged 1 commit into
ggml-org:masterfrom
dfriehs:cuda-warn-unused
Aug 7, 2026
Merged

cuda: fix warnings for unused variable/function#26688
ggerganov merged 1 commit into
ggml-org:masterfrom
dfriehs:cuda-warn-unused

Conversation

@dfriehs

@dfriehs dfriehs commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Overview

Fixes warnings when compiling with GGML_CUDA_NO_PEER_COPY or for GPUs below Blackwell.

Additional information

nvfp4_native_scale_error is only used inside quantize_mmq_nvfp4 which is Blackwell only (also surrounded by defined(BLACKWELL_MMA_AVAILABLE)).

Requirements

@dfriehs
dfriehs requested a review from a team as a code owner August 6, 2026 19:38
@github-actions github-actions Bot added ggml changes relating to the ggml tensor library for machine learning CUDA Related to the CUDA backend labels Aug 6, 2026
@am17an am17an added the merge ready A maintainer can use this label to indicate that they consider the changes final and ready to merge. label Aug 7, 2026
@ggerganov
ggerganov merged commit 5b87ed3 into ggml-org:master Aug 7, 2026
1 check passed
@dfriehs
dfriehs deleted the cuda-warn-unused branch August 7, 2026 16:41
miltos22 pushed a commit to miltos22/llama.cpp-wackMall-merge-request that referenced this pull request Aug 10, 2026
satindergrewal pushed a commit to satindergrewal/llama.cpp that referenced this pull request Aug 11, 2026
satindergrewal pushed a commit to satindergrewal/llama.cpp that referenced this pull request Aug 12, 2026
huaxel pushed a commit to huaxel/CachyLLama that referenced this pull request Aug 12, 2026
brittlewis12 pushed a commit to brittlewis12/llama.cpp that referenced this pull request Aug 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CUDA Related to the CUDA backend ggml changes relating to the ggml tensor library for machine learning merge ready A maintainer can use this label to indicate that they consider the changes final and ready to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants