Skip to content

Add gfx950 #806

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jun 11, 2025
Merged

Add gfx950 #806

merged 2 commits into from
Jun 11, 2025

Conversation

marbre
Copy link
Member

@marbre marbre commented Jun 10, 2025

No description provided.

@@ -11,6 +11,14 @@
"expect_failure": True,
}
},
"gfx950": {
Copy link
Member

Choose a reason for hiding this comment

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

Can this be tested on the PR itself using a github label to opt-in to this build? Or are you testing with workflow_dispatch somewhere?

Copy link
Member Author

Choose a reason for hiding this comment

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

I raised this in internal chat as I am not sure but wanted to send this for early feedback.

Copy link
Member

Choose a reason for hiding this comment

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

By "tested" I mean test that the build actually succeeds/fails as indicated by expect_failure, not "tested" as in runs tests on a machine with a GPU.

Copy link
Member Author

Choose a reason for hiding this comment

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

With "test-runs-on": "", it should only build and not run tests for now but I haven't tested the former yet.

Copy link
Member

Choose a reason for hiding this comment

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

Yep that's fine. I'm just interested in knowing if the build succeeds (sorry for the overloaded "tested" term)

@marbre
Copy link
Member Author

marbre commented Jun 10, 2025

The manually triggered build for gfx950 passed: https://github.com/ROCm/TheRock/actions/runs/15568943706/job/43839877992

Hence, marked this not expected to fail.

@marbre marbre marked this pull request as ready for review June 10, 2025 22:58
@marbre marbre requested a review from geomin12 June 10, 2025 22:59
@@ -13,6 +13,13 @@
"pytorch-target": "gfx942",
}
},
"gfx950": {
Copy link
Contributor

Choose a reason for hiding this comment

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

is this gfx95x? or gfx950?

Copy link
Contributor

Choose a reason for hiding this comment

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

i can add a label, so folks can add that label and run that target for PRs

@stellaraccident stellaraccident merged commit c144532 into main Jun 11, 2025
17 checks passed
@stellaraccident stellaraccident deleted the users/marbre/gfx950 branch June 11, 2025 03:52
@github-project-automation github-project-automation bot moved this from TODO to Done in TheRock Triage Jun 11, 2025
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.

4 participants