Skip to content

Handle explicit empty target selection#2678

Merged
bsyrowik merged 1 commit into
developfrom
users/bsyrowik/rocwmma_explicit_blank_target
Nov 17, 2025
Merged

Handle explicit empty target selection#2678
bsyrowik merged 1 commit into
developfrom
users/bsyrowik/rocwmma_explicit_blank_target

Conversation

@bsyrowik
Copy link
Copy Markdown
Contributor

Motivation

Long-term rocWMMA fix for ROCm/TheRock#2136

Technical Details

Test Plan

Test Result

Submission Checklist

Copy link
Copy Markdown
Contributor

@evedovelli evedovelli left a comment

Choose a reason for hiding this comment

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

lgtm

@bsyrowik bsyrowik merged commit 5a7c04b into develop Nov 17, 2025
56 of 65 checks passed
@bsyrowik bsyrowik deleted the users/bsyrowik/rocwmma_explicit_blank_target branch November 17, 2025 16:35
assistant-librarian Bot pushed a commit to ROCm/rocWMMA that referenced this pull request Nov 17, 2025
[rocWMMA] Handle explicit empty target selection

## Motivation

Long-term rocWMMA fix for ROCm/TheRock#2136

## Technical Details

<!-- Explain the changes along with any relevant GitHub links. -->

## Test Plan

<!-- Explain any relevant testing done to verify this PR. -->

## Test Result

<!-- Briefly summarize test outcomes. -->

## Submission Checklist

- [ ] Look over the contributing guidelines at
https://github.com/ROCm/ROCm/blob/develop/CONTRIBUTING.md#pull-requests.
bsyrowik added a commit to ROCm/TheRock that referenced this pull request Nov 24, 2025
## Motivation

Remove the workaround for when we try to build rocWMMA with an empty
target.

## Technical Details

As of 2025/11/19 rocWMMA will default to the first compatible target in
the case where we explicitly specify an empty GPU_TARGET. See
ROCm/rocm-libraries#2678

## Test Plan

Test against gfx103X:
https://github.com/ROCm/TheRock/actions/runs/19517010601

<!-- Explain any relevant testing done to verify this PR. -->

## Test Result

<!-- Briefly summarize test outcomes. -->

## Submission Checklist

- [ ] Look over the contributing guidelines at
https://github.com/ROCm/ROCm/blob/develop/CONTRIBUTING.md#pull-requests.
bsyrowik added a commit that referenced this pull request Nov 26, 2025
## Motivation

Long-term rocWMMA fix for ROCm/TheRock#2136

## Technical Details

<!-- Explain the changes along with any relevant GitHub links. -->

## Test Plan

<!-- Explain any relevant testing done to verify this PR. -->

## Test Result

<!-- Briefly summarize test outcomes. -->

## Submission Checklist

- [ ] Look over the contributing guidelines at
https://github.com/ROCm/ROCm/blob/develop/CONTRIBUTING.md#pull-requests.
@bsyrowik bsyrowik mentioned this pull request Nov 26, 2025
1 task
rponnuru5 pushed a commit to ROCm/TheRock that referenced this pull request Dec 9, 2025
## Motivation

Remove the workaround for when we try to build rocWMMA with an empty
target.

## Technical Details

As of 2025/11/19 rocWMMA will default to the first compatible target in
the case where we explicitly specify an empty GPU_TARGET. See
ROCm/rocm-libraries#2678

## Test Plan

Test against gfx103X:
https://github.com/ROCm/TheRock/actions/runs/19517010601

<!-- Explain any relevant testing done to verify this PR. -->

## Test Result

<!-- Briefly summarize test outcomes. -->

## Submission Checklist

- [ ] Look over the contributing guidelines at
https://github.com/ROCm/ROCm/blob/develop/CONTRIBUTING.md#pull-requests.
ammallya pushed a commit that referenced this pull request Feb 3, 2026
* Rename vector to ThreadTile

* more notes on tile encoding

* remove number<> from tuple of make_tile_window

* add script to stress test the copy example

[ROCm/composable_kernel commit: 8a698c7]
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