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

Fix FP8 and BF16 benchmark output - #20

Merged
cfallows-amd merged 2 commits into
mainfrom
cfallows-amd/fp8fix
Oct 22, 2025
Merged

Fix FP8 and BF16 benchmark output#20
cfallows-amd merged 2 commits into
mainfrom
cfallows-amd/fp8fix

Conversation

@cfallows-amd

@cfallows-amd cfallows-amd commented Oct 21, 2025

Copy link
Copy Markdown
Collaborator

Motivation

Remove FP8 and BF16 peak VALU measurements.

Technical Details

FP8 is a data type optimized for mfma support and do not actually run on the VALU unit. FP8 should only have a peak MFMA roofline value; we are removing the peak VALU FP8 value.
Similarly, there is no BF16 add/mul VOP.

Test Plan

Manual build and run of roofline binary on various distros- rhel8, sles15sp6, ubuntu22.04, azl3.

Test Result

FP8 and BF16 peak VALU is removed from the CLI output as well as the roofline csv.

Submission Checklist

Signed-off-by: Carrie Fallows <Carrie.Fallows@amd.com>
@cfallows-amd
cfallows-amd requested a review from a team as a code owner October 21, 2025 18:54
Signed-off-by: Carrie Fallows <Carrie.Fallows@amd.com>
@cfallows-amd cfallows-amd changed the title Fix FP8 benchmark output Fix FP8 and BF16 benchmark output Oct 22, 2025
@cfallows-amd
cfallows-amd merged commit 2113ef1 into main Oct 22, 2025
2 of 4 checks passed
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.

2 participants