Skip to content

Develop branch for release after ROCm 7.0#1022

Merged
daineAMD merged 1 commit into
ROCm:developfrom
daineAMD:71dev
Jul 18, 2025
Merged

Develop branch for release after ROCm 7.0#1022
daineAMD merged 1 commit into
ROCm:developfrom
daineAMD:71dev

Conversation

@daineAMD
Copy link
Copy Markdown
Contributor

hipblas-common dep. for release-staging/rocm-rel-7.0 branch is v 1.1 but should probably be changed to 1.2 to be consistent. I'll make a change to that branch unless anyone thinks otherwise.

Pending rocsolver version and hipblas-common merge.

Copy link
Copy Markdown
Contributor

@TorreZuk TorreZuk left a comment

Choose a reason for hiding this comment

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

Versions look fine with listed caveat

Comment thread CMakeLists.txt
rocm_package_add_deb_dependencies(STATIC_DEPENDS "rocblas-static-dev >= ${rocblas_minimum}" "rocsolver-static-dev >= ${rocsolver_minimum}")
endif( )

set(hipblas_common_minimum 1.1.0)
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.

Well for dependencies not sure if we have a tight or loose policy for minimum version. Think solver must be loose as they always talk about using older rocblas too

@daineAMD daineAMD merged commit 26edbf2 into ROCm:develop Jul 18, 2025
25 of 29 checks passed
@daineAMD daineAMD deleted the 71dev branch July 18, 2025 19:36
ammallya pushed a commit that referenced this pull request Oct 28, 2025
ammallya added a commit that referenced this pull request Oct 28, 2025
Add Clang guards around usage of openmp config (#1050)
Develop branch for release after ROCm 7.0 (#1022)
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