Skip to content
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

Add current generation CPUs and GPUs. #764

Merged
merged 4 commits into from
Oct 4, 2024

Conversation

garloff
Copy link
Member

@garloff garloff commented Sep 26, 2024

So we add AMD Zen-5/5c, intel Sierra Forest (Crestmont) and Granite Rapids (Redwood Cove), ARM AmpereOne and Cortex-72x/NeoverseN3/V3.

On the GPU side, add nVidia Hopper (h), C/RDNA 3.5 (3.5) and C/RDNA4 (4), intel Xe12.2 (2) and Arc/12.7/DG2 (3).

Added it to the spec and the code.
Spec does not get a new revision number, as this is a fully backwards- compatible enahncement (and logical continuation). We could of course update the minor version (and call this v3.2).

So we add AMD Zen-5/5c, intel Sierra Forest (Crestmont) and Granite
Rapids (Redwood Cove), ARM AmpereOne and Cortex-72x/NeoverseN3/V3.

On the GPU side, add nVidia Hopper (h), C/RDNA 3.5 (3.5) and C/RDNA4 (4),
intel Xe12.2 (2) and Arc/12.7/DG2 (3).

Added it to the spec and the code.
Spec does not get a new revision number, as this is a fully backwards-
compatible enahncement (and logical continuation). We could of course
update the minor version (and call this v3.2).

Signed-off-by: Kurt Garloff <[email protected]>
@garloff garloff added enhancement New feature or request standards Issues / ADR / pull requests relevant for standardization & certification labels Sep 26, 2024
@garloff garloff self-assigned this Sep 26, 2024
And our parser does exactly this.
So let's use 'g' for GraceHopper.

Signed-off-by: Kurt Garloff <[email protected]>
We call these "SMs/CUs/EUs" for nVidia/AMD/intel, in the order of
market penetration for these GPUs and the order they are listed
in the standard. Improves readability a tiny bit ...

Signed-off-by: Kurt Garloff <[email protected]>
Copy link
Contributor

@mbuechse mbuechse left a comment

Choose a reason for hiding this comment

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

To be frank, this seems a bit messy to me, which is a testament to the fact that flavor naming is a crutch, but I won't object because that seems to be the state of the art for better or worse...

@garloff garloff merged commit d0dce6d into main Oct 4, 2024
7 checks passed
@garloff garloff deleted the feat/flavor-names-new-cpu-gpu-gens branch October 4, 2024 06:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request standards Issues / ADR / pull requests relevant for standardization & certification
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants