Skip to content

rocrtst: Support GPU targets defined in theRock#3400

Merged
JeniferC99 merged 1 commit intodevelopfrom
users/alysaliu/AILIROCR-35
Feb 24, 2026
Merged

rocrtst: Support GPU targets defined in theRock#3400
JeniferC99 merged 1 commit intodevelopfrom
users/alysaliu/AILIROCR-35

Conversation

@amd-alysaliu
Copy link
Copy Markdown
Contributor

Use GPU_TARGETS/AMDGPU_TARGETS defined in TheRock when TARGET_DEVICES not set.

Related to issue #2262

@amd-alysaliu amd-alysaliu force-pushed the users/alysaliu/AILIROCR-35 branch 2 times, most recently from 404f3d7 to 35a51f9 Compare February 20, 2026 16:21
Copy link
Copy Markdown
Contributor

@shwetagkhatri shwetagkhatri left a comment

Choose a reason for hiding this comment

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

LGTM

Use GPU_TARGETS/AMDGPU_TARGETS defined in TheRock when TARGET_DEVICES not set.

Signed-off-by: Alysa Liu <Alysa.Liu@amd.com>
@amd-alysaliu amd-alysaliu force-pushed the users/alysaliu/AILIROCR-35 branch from 35a51f9 to bad086b Compare February 23, 2026 15:21
@JeniferC99
Copy link
Copy Markdown
Collaborator

passing ocl and hip: http://rocm-ci.amd.com/job/rocm-tests/40967/

@JeniferC99 JeniferC99 merged commit e6efe21 into develop Feb 24, 2026
19 of 20 checks passed
@JeniferC99 JeniferC99 deleted the users/alysaliu/AILIROCR-35 branch February 24, 2026 17:46
amd-josnarlo pushed a commit that referenced this pull request Feb 25, 2026
Use GPU_TARGETS/AMDGPU_TARGETS defined in TheRock when TARGET_DEVICES not set.

Signed-off-by: Alysa Liu <Alysa.Liu@amd.com>
radhaksri pushed a commit that referenced this pull request Feb 26, 2026
Use GPU_TARGETS/AMDGPU_TARGETS defined in TheRock when TARGET_DEVICES not set.

Signed-off-by: Alysa Liu <Alysa.Liu@amd.com>
@jayhawk-commits
Copy link
Copy Markdown
Collaborator

jayhawk-commits commented Feb 27, 2026

rocr test failures in ROCm/TheRock#3602 where bin files are not found are potentially from this PR #3400. See log: https://github.com/ROCm/TheRock/actions/runs/22430784397/job/65077513396?pr=3602

The test script in TheRock for executing rocr tests need to be updated first. @jharryma

This gap would not have been caught through rocm-systems pull requests until #3182 lands. @rahulc-gh

The changes in this PR #3400 need to land on TheRock through a submodule update at the same time as the test scripts for ROCR are updated to account for the bin changes.

jharryma added a commit to ROCm/TheRock that referenced this pull request Mar 3, 2026
## Motivation

Resolves: #3666

rocm-systems submodule bump #3668
introduced an issue on TheRock CI with the rocrtst generic tarball where
only the last gfx arch built rocrtst binaries are the ones uploaded to
the generic tarball. (e.g. if gfx120X tarball is built last, it will be
the only one included in the generic tarball, causing gfx94X tests to
fail)

Introduced in this rocm-systems PR:
ROCm/rocm-systems#3400

This issue is similar to aqlprofile and hip-tests:
#3679. We'll need to set it back
to target-neutral for multi-arch CI

## Technical Details

Setting rocrtst to target-specific. 

## Test Plan

Test rocrtst for multiple archs

## Test Result

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

## Submission Checklist

- [x] 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants