Skip to content

Conversation

@umfranzw
Copy link
Collaborator

@umfranzw umfranzw commented May 8, 2025

We are now using some newer google benchmark features that are not present in 1.8.0. Update the cmake dependencies file to pull version 1.9.0.

NguyenNhuDi
NguyenNhuDi approved these changes May 8, 2025
Copy link
Contributor

@NguyenNhuDi NguyenNhuDi left a comment

Choose a reason for hiding this comment

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

This looks good to me. But maybe it's worthwhile to update the changelog as well. I think there is a change in the default units displayed in "console" mode when going from version 1.8.0 to version 1.9.0 (MB/s, GB/s, TB/s to Mi/s, Gi/s, Ti/s) .

We are now using some newer google benchmark features that are
not present in 1.8.0. Update the cmake dependencies file to
pull version 1.9.0.
@umfranzw umfranzw force-pushed the update_gbench_version branch from 2baa844 to 20014ff Compare May 9, 2025 19:23
@umfranzw umfranzw requested a review from a team as a code owner May 9, 2025 19:23
@umfranzw
Copy link
Collaborator Author

umfranzw commented May 9, 2025

Thanks @NguyenNhuDi - I've added a note about this to the changelog.

@umfranzw umfranzw requested a review from NguyenNhuDi May 9, 2025 19:24
Copy link
Contributor

@NguyenNhuDi NguyenNhuDi left a comment

Choose a reason for hiding this comment

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

lgtm. Thanks Wayne

@umfranzw umfranzw merged commit cf22cb3 into ROCm:develop May 16, 2025
5 of 18 checks passed
umfranzw added a commit to umfranzw/rocThrust that referenced this pull request Jun 4, 2025
We are now using some newer google benchmark features that are
not present in 1.8.0. Update the cmake dependencies file to
pull version 1.9.0.
shahamed pushed a commit that referenced this pull request Jun 11, 2025
We are now using some newer google benchmark features that are
not present in 1.8.0. Update the cmake dependencies file to
pull version 1.9.0.
jharryma pushed a commit that referenced this pull request Jun 12, 2025
We are now using some newer google benchmark features that are
not present in 1.8.0. Update the cmake dependencies file to
pull version 1.9.0.
ammallya pushed a commit that referenced this pull request Oct 28, 2025
We are now using some newer google benchmark features that are
not present in 1.8.0. Update the cmake dependencies file to
pull version 1.9.0.

[ROCm/rocThrust commit: cf22cb3]
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.

4 participants