Skip to content

perf(hipblaslt): update gfx942 GridBased BSS NT liblogic file by reassigning solution indices - #13368

Merged
shbae merged 1 commit into
developfrom
users/sbae/update-bf16-gemm-MI300X-liblogic
Oct 10, 2026
Merged

shbae merged 1 commit into
developfrom
users/sbae/update-bf16-gemm-MI300X-liblogic

Conversation

@shbae

@shbae shbae commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

JIRA ID: https://amd-hub.atlassian.net/browse/AIROCBLAS-1267

Motivation

As part of working on performance parity effort for rocblas-bench with hipBLASLt backend for BF16, the change in this PR reduces the number of regressed cases for BSS ant NT operation type by applying --algo_method all optimization.

Technical Details

After running --algo-method all for the grid-points that are in regressed cases, we reassign the best GridBased solution index for each grid-point. This results in reduces the performance regressed cases w.r.t. Tensile and hipBLASLt backend comparison.

Test Result

BSS, NT Case: Baseline vs. Opt

BSS datatype — Total tests: 3,162

Regression Analysis

Metric Baseline Opt
Regressed cases (count) 174 146
Regressed cases (%) 5.50% 4.62%

Backend Timing (seconds)

Metric Baseline Opt
Total time – Tensile backend (sec.) 13.50 (same for both — unaffected by optimization)
Total time – hipBLASLt backend (sec.) 9.05 8.74

Overall Improvement with Opt

Metric Baseline Opt
Improved cases (>1.05x speedup), % 91.11% 91.62%
Geomean speedup 1.76x 1.81x

Note: BSS, NT case was already strong at baseline (5.50% regressed, 1.76x geomean). The optimization trims the regressed rate further to 4.62% and lifts the geomean speedup to 1.81x.

Submission Checklist

@therock-pr-bot

therock-pr-bot Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

✅ All Checks Passed — Ready for Review

Check Status Details
📝 PR Description ✅ Pass —
⛔ Forbidden Files ✅ Pass —
🧪 Unit Test ✅ Pass PR does not contain code files — Unit Test auto-passed
🔎 pre-commit ✅ Pass —
🚫 Draft PR 🔜 To Be Enabled —
🚩 Feature Flag 🔜 To Be Enabled —
📊 Code Coverage 🔜 To Be Enabled —
🤖 therock-pr-bot ✅ Pass —

🎉 All checks passed! This PR is ready for review.

📖 Need help? See the Policy FAQ for details on every check and how to fix failures.

🙋 Wish to Override Policy?

@therock-pr-bot

therock-pr-bot Bot commented Oct 8, 2026

Copy link
Copy Markdown

🎉 All checks passed! This PR is ready for review.

@codecov

codecov Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #13368      +/-   ##
===========================================
- Coverage    72.16%   72.16%   -0.00%     
===========================================
  Files         2874     2874              
  Lines       474873   474873              
  Branches     70141    70141              
===========================================
- Hits        342674   342673       -1     
  Misses      108398   108398              
- Partials     23801    23802       +1     
Flag Coverage Δ *Carryforward flag
TensileLite-CPP 49.33% <ø> (ø)
TensileLite-Unit 83.33% <ø> (ø)
hipBLAS 90.62% <ø> (ø) Carriedforward from faea76a
hipBLASLt 41.96% <ø> (-<0.01%) ⬇️
hipCUB 82.68% <ø> (ø) Carriedforward from faea76a
hipDNN 86.66% <ø> (ø) Carriedforward from faea76a
hipFFT 42.52% <ø> (ø) Carriedforward from faea76a
hipRAND 76.12% <ø> (ø) Carriedforward from faea76a
hipSOLVER 68.96% <ø> (ø) Carriedforward from faea76a
hipSPARSE 86.99% <ø> (ø) Carriedforward from faea76a
rocBLAS 48.17% <ø> (ø) Carriedforward from faea76a
rocFFT 51.45% <ø> (ø) Carriedforward from faea76a
rocRAND 57.45% <ø> (ø) Carriedforward from faea76a
rocSOLVER 76.83% <ø> (ø) Carriedforward from faea76a
rocSPARSE 74.07% <ø> (ø) Carriedforward from faea76a
rocThrust 93.15% <ø> (ø) Carriedforward from faea76a

*This pull request uses carry forward flags. Click here to find out more.
see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@shbae
shbae requested a review from nakajee October 9, 2026 18:58

@nakajee nakajee 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.

Looks good

@shbae
shbae merged commit c299413 into develop Oct 10, 2026
129 of 142 checks passed
@shbae
shbae deleted the users/sbae/update-bf16-gemm-MI300X-liblogic branch October 10, 2026 03:25
@assistant-librarian

Copy link
Copy Markdown
Contributor

TheRock Submodule Bump Activity

Newest first

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants