Skip to content

[hipblsalt] Fix leaky hip::device dependency#855

Merged
davidd-amd merged 1 commit into
developfrom
users/ellosel/hipblaslt-hotfix-device-target
Jul 26, 2025
Merged

[hipblsalt] Fix leaky hip::device dependency#855
davidd-amd merged 1 commit into
developfrom
users/ellosel/hipblaslt-hotfix-device-target

Conversation

@davidd-amd
Copy link
Copy Markdown
Contributor

@davidd-amd davidd-amd commented Jul 25, 2025

Copy link
Copy Markdown
Contributor

@stellaraccident stellaraccident left a comment

Choose a reason for hiding this comment

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

Thanks!

@davidd-amd
Copy link
Copy Markdown
Contributor Author

This only changes next-cmake dir - merging.

@davidd-amd davidd-amd merged commit e056cac into develop Jul 26, 2025
9 of 12 checks passed
@davidd-amd davidd-amd deleted the users/ellosel/hipblaslt-hotfix-device-target branch July 26, 2025 01:44
assistant-librarian Bot pushed a commit to ROCm/hipBLASLt that referenced this pull request Jul 26, 2025
[hipblsalt] Fix leaky hip::device dependency

- Do note use PUBLIC linkage for hip::device
- Add logic to project config to catch when hip::device is added to
interface link libraries
- Fixes the regression noted in
ROCm/rocm-libraries#650 (comment)
- Root cause diagnosis of the issue:
https://chatgpt.com/share/68841788-4564-800d-a6a1-e000c668c0fb
ScottTodd added a commit to ROCm/TheRock that referenced this pull request Jul 26, 2025
Cherry-pick of ROCm/rocm-libraries#855 for
testing, with conflicts resolved manually. Patches did not apply cleanly
on a roll-up, as ROCm/rocm-libraries#702 is
still under review and has conflicts too.

Testing at:
* Linux gfx942: https://github.com/ROCm/TheRock/actions/runs/16535937296
* Windows gfx110X-dgpu:
https://github.com/ROCm/TheRock/actions/runs/16535942310
dsaffars pushed a commit to dsaffars/rocm-libraries that referenced this pull request Jul 28, 2025
Bumps [rocm-docs-core](https://github.com/ROCm/rocm-docs-core) from 1.8.1 to 1.8.2.
- [Release notes](https://github.com/ROCm/rocm-docs-core/releases)
- [Changelog](https://github.com/ROCm/rocm-docs-core/blob/v1.8.2/CHANGELOG.md)
- [Commits](ROCm/rocm-docs-core@v1.8.1...v1.8.2)

---
updated-dependencies:
- dependency-name: rocm-docs-core
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

[ROCm/rocSPARSEcommit: 0015150]
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