Skip to content

UPSTREAM PR #18443: ggml-cuda: (cmake) expand "native" into concrete architectures#735

Open
loci-dev wants to merge 1 commit intomainfrom
upstream-PR18443-branch_QDelta-master
Open

UPSTREAM PR #18443: ggml-cuda: (cmake) expand "native" into concrete architectures#735
loci-dev wants to merge 1 commit intomainfrom
upstream-PR18443-branch_QDelta-master

Conversation

@loci-dev
Copy link
Copy Markdown

Mirrored from ggml-org/llama.cpp#18443

Fixes #18430 and possibly #18434.

#18413 caused error on blackwell local build because native will be treated as 120-real but is not captured by the regex.

Tested with or without docker (with a blackwell GPU). Seems OK.

@loci-review
Copy link
Copy Markdown

loci-review bot commented Dec 29, 2025

Explore the complete analysis inside the Version Insights

Perfect! I've generated the summary report for your project. Here's what the analysis shows:

Key Findings:

✅ No Performance Issues Detected

The performance analysis of Pull Request #735 in the llama.cpp repository shows that:

  • No modified functions have performance changes greater than 2% threshold
  • Both Response Time and Throughput Time metrics remain stable
  • All changes are within acceptable performance variance

Recommendation:

This pull request appears to be safe to merge from a performance perspective. The changes introduced do not cause any significant performance regressions, with all modified functions maintaining performance within 2% of the baseline version.

This is a positive result indicating that the code changes maintain the performance characteristics of the existing codebase.

@loci-dev loci-dev force-pushed the main branch 27 times, most recently from ac31769 to 7aa8b1c Compare January 1, 2026 12:15
@loci-dev loci-dev force-pushed the main branch 30 times, most recently from 1de1e20 to 4be6e0f Compare January 7, 2026 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants