Skip to content

Remove rocWMMA empty target workaround#2212

Merged
bsyrowik merged 1 commit into
mainfrom
users/bsyrowik/rocwmma_empty_target
Nov 24, 2025
Merged

Remove rocWMMA empty target workaround#2212
bsyrowik merged 1 commit into
mainfrom
users/bsyrowik/rocwmma_empty_target

Conversation

@bsyrowik
Copy link
Copy Markdown
Contributor

@bsyrowik bsyrowik commented Nov 19, 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

Test Result

Submission Checklist

@bsyrowik bsyrowik force-pushed the users/bsyrowik/rocwmma_empty_target branch from 62239a5 to 5c15612 Compare November 20, 2025 22:47
@bsyrowik bsyrowik marked this pull request as ready for review November 20, 2025 22:48
@bsyrowik bsyrowik merged commit 34c8147 into main Nov 24, 2025
58 checks passed
@bsyrowik bsyrowik deleted the users/bsyrowik/rocwmma_empty_target branch November 24, 2025 16:18
@github-project-automation github-project-automation Bot moved this from TODO to Done in TheRock Triage Nov 24, 2025
rponnuru5 pushed a commit 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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants