Skip to content

fix(hipblaslt): Load the RNDseed value into the sgpr when using software rounding - #14337

Merged
pmckinno merged 1 commit into
developfrom
users/pmckinno/stochastic_rounding_mi300
Oct 26, 2026
Merged

pmckinno merged 1 commit into
developfrom
users/pmckinno/stochastic_rounding_mi300

Conversation

@pmckinno

Copy link
Copy Markdown
Collaborator

##Motivation

On architectures without v_prng_b32, the random seed was defined but never written.

JIRA ID: AIHPBLAS-5267

@pmckinno
pmckinno requested a review from a team as a code owner October 23, 2026 22:01
@pmckinno
pmckinno force-pushed the users/pmckinno/stochastic_rounding_mi300 branch 2 times, most recently from a87c248 to 1ba9785 Compare October 23, 2026 22:05
@therock-pr-bot

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

Copy link
Copy Markdown

✅ All Checks Passed — Ready for Review

Check Status Details
📝 PR Description ✅ Pass —
⛔ Forbidden Files ✅ Pass —
🧪 Unit Test ✅ Pass —
🔎 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

Copy link
Copy Markdown

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

@pmckinno
pmckinno force-pushed the users/pmckinno/stochastic_rounding_mi300 branch from 1ba9785 to ff09e3d Compare October 23, 2026 23:07
@codecov

codecov Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Additional details and impacted files
@@            Coverage Diff            @@
##           develop   #14337    +/-   ##
=========================================
  Coverage    75.91%   75.91%            
=========================================
  Files         2883     2884     +1     
  Lines       480800   480964   +164     
  Branches     72335    72368    +33     
=========================================
+ Hits        364994   365122   +128     
- Misses       90769    90798    +29     
- Partials     25037    25044     +7     
Flag Coverage Δ *Carryforward flag
TensileLite-CPP 48.93% <ø> (ø)
TensileLite-Unit 83.71% <100.00%> (-0.01%) ⬇️
hipBLAS 90.62% <ø> (ø) Carriedforward from 901e5df
hipBLASLt 66.18% <ø> (-0.02%) ⬇️
hipCUB 82.68% <ø> (ø) Carriedforward from 901e5df
hipDNN 86.84% <ø> (ø) Carriedforward from 901e5df
hipFFT 42.56% <ø> (ø) Carriedforward from 901e5df
hipRAND 76.12% <ø> (ø) Carriedforward from 901e5df
hipSOLVER 69.67% <ø> (ø) Carriedforward from 901e5df
hipSPARSE 86.99% <ø> (ø) Carriedforward from 901e5df
rocBLAS 68.72% <ø> (ø) Carriedforward from 901e5df
rocFFT 46.05% <ø> (ø) Carriedforward from 901e5df
rocRAND 57.31% <ø> (ø) Carriedforward from 901e5df
rocSOLVER 76.74% <ø> (ø) Carriedforward from 901e5df
rocSPARSE 77.16% <ø> (ø) Carriedforward from 901e5df
rocThrust 93.15% <ø> (ø) Carriedforward from 901e5df

*This pull request uses carry forward flags. Click here to find out more.

Files with missing lines Coverage Δ
...lt/tensilelite/tensilelite/Components/Signature.py 95.93% <100.00%> (+0.04%) ⬆️
.../hipblaslt/tensilelite/tensilelite/Contractions.py 85.92% <100.00%> (+0.02%) ⬆️
.../hipblaslt/tensilelite/tensilelite/KernelWriter.py 84.81% <100.00%> (+0.01%) ⬆️
...lt/tensilelite/tensilelite/KernelWriterAssembly.py 79.73% <100.00%> (+<0.01%) ⬆️

... and 17 files 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.

@pmckinno
pmckinno force-pushed the users/pmckinno/stochastic_rounding_mi300 branch from ff09e3d to 48f1598 Compare October 26, 2026 15:31
…are rounding

On architectures without v_prng_b32, the random seed was defined but never written.

JIRA ID: AIHPBLAS-5267
@pmckinno
pmckinno force-pushed the users/pmckinno/stochastic_rounding_mi300 branch from 48f1598 to 06733d4 Compare October 26, 2026 15:44
@pmckinno
pmckinno merged commit e6b2505 into develop Oct 26, 2026
166 of 170 checks passed
@pmckinno
pmckinno deleted the users/pmckinno/stochastic_rounding_mi300 branch October 26, 2026 23:03
@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.

3 participants