Skip to content

refactor(hipsparse): retire 4.8 feature flags - #14144

Merged
amontoison merged 1 commit into
ROCm:developfrom
amontoison:hipsparse_remove_old_feature_flags
Oct 23, 2026
Merged

amontoison merged 1 commit into
ROCm:developfrom
amontoison:hipsparse_remove_old_feature_flags

Conversation

@amontoison

Copy link
Copy Markdown
Contributor

Summary

  • Retires the hipSPARSE SPGEAM build flag, which have been added in the 4.8.0 release.
  • Removes the HIPSPARSE_ENABLE_SPGEAM CMake option, the HIPSPARSE_WITH_SPGEAM macro in hipsparse-config.h.in and its Doxygen PREDEFINED entry.
  • hipsparse-config.h.in and the CheckFeatureFlags guardrail are kept for future flags.

Companion of #14137

Remove the HIPSPARSE_ENABLE_SPGEAM option and the HIPSPARSE_WITH_SPGEAM
config macro; the generic SpGEAM API is now always built (still gated on
CUDA >= 13.3 for the NVIDIA backend).
@therock-pr-bot

therock-pr-bot Bot commented Oct 21, 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.

@codecov

codecov Bot commented Oct 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop   #14144   +/-   ##
========================================
  Coverage    75.94%   75.94%           
========================================
  Files         2873     2873           
  Lines       477013   477013           
  Branches     71574    71574           
========================================
  Hits        362228   362228           
  Misses       90089    90089           
  Partials     24696    24696           
Flag Coverage Δ *Carryforward flag
TensileLite-CPP 48.93% <ø> (ø) Carriedforward from 9ce95cd
TensileLite-Unit 83.65% <ø> (ø) Carriedforward from 9ce95cd
hipBLAS 90.62% <ø> (ø) Carriedforward from 9ce95cd
hipBLASLt 64.43% <ø> (ø) Carriedforward from 9ce95cd
hipCUB 82.68% <ø> (ø) Carriedforward from 9ce95cd
hipDNN 87.04% <ø> (ø) Carriedforward from 9ce95cd
hipFFT 42.00% <ø> (ø) Carriedforward from 9ce95cd
hipRAND 76.12% <ø> (ø) Carriedforward from 9ce95cd
hipSOLVER 69.67% <ø> (ø) Carriedforward from 9ce95cd
hipSPARSE 86.99% <ø> (ø)
rocBLAS 68.73% <ø> (ø) Carriedforward from 9ce95cd
rocFFT 48.10% <ø> (ø) Carriedforward from 9ce95cd
rocRAND 57.30% <ø> (ø) Carriedforward from 9ce95cd
rocSOLVER 76.74% <ø> (ø) Carriedforward from 9ce95cd
rocSPARSE 76.92% <ø> (ø) Carriedforward from 9ce95cd
rocThrust 93.15% <ø> (ø) Carriedforward from 9ce95cd

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

Files with missing lines Coverage Δ
...ibrary/src/amd_detail/generic/hipsparse_spgeam.cpp 95.81% <ø> (ø)
🚀 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.

@amontoison
amontoison merged commit 4e16aa6 into ROCm:develop Oct 23, 2026
140 of 147 checks passed
@amontoison
amontoison deleted the hipsparse_remove_old_feature_flags branch October 23, 2026 19:38
@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