Skip to content

rocm: add all GFX9, GFX10 and GFX11 amdgpu_targets#33871

Merged
tldahlgren merged 1 commit intospack:developfrom
cgmb:extend-amdgpu-targets
Nov 14, 2022
Merged

rocm: add all GFX9, GFX10 and GFX11 amdgpu_targets#33871
tldahlgren merged 1 commit intospack:developfrom
cgmb:extend-amdgpu-targets

Conversation

@cgmb
Copy link
Copy Markdown
Contributor

@cgmb cgmb commented Nov 13, 2022

This change adds all documented AMDGPU processors from GFX9 through GFX11 and sorts the list.

There have been proposals such as #29018 which have been more ambitious about improving the amdgpu_targets. I'm cheering that effort on, but for the moment maybe we can just extend the list with the newly added values.

@spackbot-app spackbot-app bot added build-systems core PR affects Spack core functionality labels Nov 13, 2022
@cgmb cgmb changed the title Add more amdgpu_targets rocm: add more amdgpu_targets Nov 13, 2022
This change adds all documented AMDGPU processors from GFX9 through
GFX11 and sorts the list.
@cgmb cgmb force-pushed the extend-amdgpu-targets branch from 9ffed57 to b74ab85 Compare November 13, 2022 08:43
@cgmb cgmb changed the title rocm: add more amdgpu_targets rocm: add all GFX9, GFX10 and GFX11 amdgpu_targets Nov 13, 2022
@dtaller
Copy link
Copy Markdown
Contributor

dtaller commented Nov 14, 2022

Thanks for keeping things up to date! I appreciate it

Copy link
Copy Markdown
Contributor

@dtaller dtaller left a comment

Choose a reason for hiding this comment

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

Looks good to me

Copy link
Copy Markdown
Contributor

@tldahlgren tldahlgren left a comment

Choose a reason for hiding this comment

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

LGTM2

@tldahlgren tldahlgren merged commit 6e39efb into spack:develop Nov 14, 2022
charmoniumQ pushed a commit to charmoniumQ/spack that referenced this pull request Nov 19, 2022
This change adds all documented AMDGPU processors from GFX9 through GFX11 and sorts the list.
amd-toolchain-support pushed a commit to amd-toolchain-support/spack that referenced this pull request Feb 16, 2023
This change adds all documented AMDGPU processors from GFX9 through GFX11 and sorts the list.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build-systems core PR affects Spack core functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants