Skip to content
This repository was archived by the owner on Jan 27, 2026. It is now read-only.

ROCm: apply specified archs to sources#237

Merged
danieldk merged 3 commits intomainfrom
rocm-per-source-arches
Oct 7, 2025
Merged

ROCm: apply specified archs to sources#237
danieldk merged 3 commits intomainfrom
rocm-per-source-arches

Conversation

@danieldk
Copy link
Member

No description provided.

Copy link
Contributor

@MekkCyber MekkCyber left a comment

Choose a reason for hiding this comment

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

Thanks for adding this !

Comment on lines 12 to +14
set(CUDA_SUPPORTED_ARCHS "{{ cuda_supported_archs }}")

set(HIP_SUPPORTED_ARCHS "gfx906;gfx908;gfx90a;gfx940;gfx941;gfx942;gfx1030;gfx1100;gfx1101")
set(HIP_SUPPORTED_ARCHS "gfx906;gfx908;gfx90a;gfx942;gfx950;gfx1030;gfx1100;gfx1101;gfx1200;gfx1201")
Copy link
Contributor

Choose a reason for hiding this comment

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

why do we not have the same thing as cuda, i mean cuda_supported_archs ?

Copy link
Member Author

Choose a reason for hiding this comment

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

How cuda_supported_archs is set up now is not ideal other. We should fix both at the same time in a different PR.

Copy link
Contributor

@MekkCyber MekkCyber left a comment

Choose a reason for hiding this comment

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

lgtm

drbh
drbh previously approved these changes Oct 1, 2025
Copy link
Collaborator

@drbh drbh left a comment

Choose a reason for hiding this comment

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

LGTM

@danieldk
Copy link
Member Author

danieldk commented Oct 1, 2025

Thanks for the reviews! Still blocked on testing this on hardware.

@danieldk danieldk force-pushed the rocm-per-source-arches branch from 584443b to 6029e51 Compare October 2, 2025 07:56
@danieldk danieldk merged commit 7b33b73 into main Oct 7, 2025
27 checks passed
@danieldk danieldk deleted the rocm-per-source-arches branch October 7, 2025 07:56
@danieldk danieldk mentioned this pull request Oct 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants