Skip to content

ggml : avoid UB in gemm ukernel + tests - #19642

Merged
ggerganov merged 1 commit into
masterfrom
gg/ggml-fa-cpu-fix-ub
Feb 15, 2026
Merged

ggerganov merged 1 commit into
masterfrom
gg/ggml-fa-cpu-fix-ub

Conversation

@ggerganov

Copy link
Copy Markdown
Member

cont #19422

  • Reword the GEMM ukernel to not trip the compiler's aggressive loop optimization warnings. It's better to avoid the global pragma as it might be useful for other static analysis
  • Add test-backend-ops with BS=75 to exercise the new tiled SIMD implementation

@ggerganov
ggerganov requested a review from am17an February 15, 2026 10:32
@github-actions github-actions Bot added testing Everything test related ggml changes relating to the ggml tensor library for machine learning labels Feb 15, 2026

@am17an am17an left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing! AFAIK the compiler warning was only tripping up the AMX backend, not sure why. But this PR is anyways more readable

@ggerganov

Copy link
Copy Markdown
Member Author

Yes these warnings maybe depend on the gcc version.

@ggerganov
ggerganov merged commit 08e6d91 into master Feb 15, 2026
76 of 78 checks passed
@ggerganov
ggerganov deleted the gg/ggml-fa-cpu-fix-ub branch February 15, 2026 12:56
liparetejas pushed a commit to liparetejas/llama.cpp that referenced this pull request Feb 23, 2026
Seunghhon pushed a commit to Seunghhon/llama.cpp that referenced this pull request Apr 26, 2026
my-other-github-account pushed a commit to my-other-github-account/llama.cpp that referenced this pull request May 15, 2026
fukuro-kun pushed a commit to fukuro-kun/fukuro-llama-cpp-turboquant that referenced this pull request Jul 5, 2026
MrLordCat referenced this pull request in MrLordCat/llama.cpp-rdna-lab Jul 16, 2026
zommiommy pushed a commit to zommiommy/llama.cpp that referenced this pull request Aug 18, 2026
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 testing Everything test related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants