Skip to content

Fix typos in VirtualWaveSize warning#495

Merged
stanleytsang-amd merged 1 commit into
ROCm:developfrom
StreamHPC:32-fix-typos-in-virtualwavesize-warning
Aug 11, 2025
Merged

Fix typos in VirtualWaveSize warning#495
stanleytsang-amd merged 1 commit into
ROCm:developfrom
StreamHPC:32-fix-typos-in-virtualwavesize-warning

Conversation

@matyas-streamhpc
Copy link
Copy Markdown
Contributor

Fix warning message from

VirtualWaveSize must be a power of two and equal or lessthan the size of hardware warp.

to

VirtualWaveSize must be a power of two and equal or less than the size of hardware warp.

@assistant-librarian assistant-librarian Bot added project: rocprim external contribution Code contribution from users community.. labels Jul 3, 2025
@matyas-streamhpc matyas-streamhpc force-pushed the 32-fix-typos-in-virtualwavesize-warning branch 3 times, most recently from a3d2ad4 to b7db489 Compare July 7, 2025 16:26
@matyas-streamhpc
Copy link
Copy Markdown
Contributor Author

Rebased.

@umfranzw umfranzw self-requested a review July 7, 2025 21:45
Copy link
Copy Markdown
Contributor

@umfranzw umfranzw left a comment

Choose a reason for hiding this comment

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

Looks good to me.

stanleytsang-amd pushed a commit to stanleytsang-amd/rocm-libraries that referenced this pull request Jul 8, 2025
* Structuring backend (ROCm#495)

* split rocsparse_handle.hpp

* split rocsparse_handle.cpp

* structuring trm_info without behaviour change

* fixing test failure related to copy_mat_info

* Fixing default initialization of rocsparse_trm_info.
Removing error prone cast when calling hipMalloc

* Fixing missing behavior check in spgeam (ROCm#514)

* Fixing missing behavior check.

* stage analysis and stage symbolic are not required for stage numeric

* spgeam final fix (ROCm#516)

* spgeam fix

---------

Co-authored-by: Yvan Mokwinski <yvan.mokwinski@gmail.com>
reboss pushed a commit to reboss/rocm-libraries that referenced this pull request Jul 8, 2025
* split rocsparse_handle.hpp

* split rocsparse_handle.cpp

* structuring trm_info without behaviour change

* fixing test failure related to copy_mat_info

* Fixing default initialization of rocsparse_trm_info.
Removing error prone cast when calling hipMalloc
@matyas-streamhpc matyas-streamhpc force-pushed the 32-fix-typos-in-virtualwavesize-warning branch from b7db489 to 12286ad Compare July 14, 2025 10:10
@matyas-streamhpc
Copy link
Copy Markdown
Contributor Author

rebased

@matyas-streamhpc matyas-streamhpc force-pushed the 32-fix-typos-in-virtualwavesize-warning branch from 12286ad to a16b63c Compare July 14, 2025 15:01
@matyas-streamhpc
Copy link
Copy Markdown
Contributor Author

rebased

ammallya pushed a commit that referenced this pull request Jul 14, 2025
* split rocsparse_handle.hpp

* split rocsparse_handle.cpp

* structuring trm_info without behaviour change

* fixing test failure related to copy_mat_info

* Fixing default initialization of rocsparse_trm_info.
Removing error prone cast when calling hipMalloc

[ROCm/rocSPARSEcommit: a2c27a3]
ammallya pushed a commit that referenced this pull request Jul 14, 2025
* Structuring backend (#495)

* split rocsparse_handle.hpp

* split rocsparse_handle.cpp

* structuring trm_info without behaviour change

* fixing test failure related to copy_mat_info

* Fixing default initialization of rocsparse_trm_info.
Removing error prone cast when calling hipMalloc

* Fixing missing behavior check in spgeam (#514)

* Fixing missing behavior check.

* stage analysis and stage symbolic are not required for stage numeric

* spgeam final fix (#516)

* spgeam fix

---------

Co-authored-by: Yvan Mokwinski <yvan.mokwinski@gmail.com>

[ROCm/rocSPARSEcommit: 22d1de0]
ammallya pushed a commit that referenced this pull request Jul 14, 2025
* Structuring backend (#495)

* split rocsparse_handle.hpp

* split rocsparse_handle.cpp

* structuring trm_info without behaviour change

* fixing test failure related to copy_mat_info

* Fixing default initialization of rocsparse_trm_info.
Removing error prone cast when calling hipMalloc

* Fixing missing behavior check in spgeam (#514)

* Fixing missing behavior check.

* stage analysis and stage symbolic are not required for stage numeric

* spgeam final fix (#516)

* spgeam fix

---------

Co-authored-by: Yvan Mokwinski <yvan.mokwinski@gmail.com>

[ROCm/rocSPARSE commit: 22d1de0]
@matyas-streamhpc matyas-streamhpc force-pushed the 32-fix-typos-in-virtualwavesize-warning branch from a16b63c to bce84b9 Compare July 15, 2025 12:38
@matyas-streamhpc
Copy link
Copy Markdown
Contributor Author

rebased

@matyas-streamhpc matyas-streamhpc force-pushed the 32-fix-typos-in-virtualwavesize-warning branch from bce84b9 to e1c33fe Compare July 16, 2025 12:35
@matyas-streamhpc
Copy link
Copy Markdown
Contributor Author

rebased

@stanleytsang-amd @RobsonRLemos

@matyas-streamhpc matyas-streamhpc force-pushed the 32-fix-typos-in-virtualwavesize-warning branch from e1c33fe to 7793aaa Compare July 21, 2025 11:35
@matyas-streamhpc matyas-streamhpc requested a review from a team as a code owner July 21, 2025 11:35
@matyas-streamhpc
Copy link
Copy Markdown
Contributor Author

rebased

ammallya pushed a commit that referenced this pull request Jul 22, 2025
Bumps [rocm-docs-core](https://github.com/ROCm/rocm-docs-core) from 1.4.1 to 1.5.0.
- [Release notes](https://github.com/ROCm/rocm-docs-core/releases)
- [Changelog](https://github.com/ROCm/rocm-docs-core/blob/develop/CHANGELOG.md)
- [Commits](ROCm/rocm-docs-core@v1.4.1...v1.5.0)

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

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

[ROCm/hipSPARSE commit: 337dea3]
@matyas-streamhpc
Copy link
Copy Markdown
Contributor Author

matyas-streamhpc commented Jul 23, 2025

rebased
@RobsonRLemos @stanleytsang-amd ?

@matyas-streamhpc matyas-streamhpc force-pushed the 32-fix-typos-in-virtualwavesize-warning branch 2 times, most recently from 1f213cc to 2406dd1 Compare July 25, 2025 12:37
@matyas-streamhpc
Copy link
Copy Markdown
Contributor Author

rebased

@matyas-streamhpc matyas-streamhpc force-pushed the 32-fix-typos-in-virtualwavesize-warning branch from 2406dd1 to ed5a921 Compare July 28, 2025 12:38
@matyas-streamhpc
Copy link
Copy Markdown
Contributor Author

rebased
@RobsonRLemos @stanleytsang-amd ?

@jayhawk-commits jayhawk-commits added the organization: streamhpc contributors from streamhpc label Jul 28, 2025
@matyas-streamhpc matyas-streamhpc force-pushed the 32-fix-typos-in-virtualwavesize-warning branch from ed5a921 to a4b3ab9 Compare August 4, 2025 12:36
@matyas-streamhpc
Copy link
Copy Markdown
Contributor Author

rebased

@stanleytsang-amd stanleytsang-amd merged commit 4048e10 into ROCm:develop Aug 11, 2025
6 of 11 checks passed
assistant-librarian Bot pushed a commit to ROCm/rocPRIM that referenced this pull request Aug 11, 2025
Fix typos in VirtualWaveSize warning

Fix warning message from

`VirtualWaveSize must be a power of two and equal or lessthan the size
of hardware warp.`

to

`VirtualWaveSize must be a power of two and equal or less than the size
of hardware warp.`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external contribution Code contribution from users community.. organization: streamhpc contributors from streamhpc project: rocprim

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants