refactor(hipblaslt): convert gfx950 HH library logic to dict format - #10391
Open
dsaffars wants to merge 1 commit into
Open
refactor(hipblaslt): convert gfx950 HH library logic to dict format#10391dsaffars wants to merge 1 commit into
dsaffars wants to merge 1 commit into
Conversation
Convert 72 gfx950 HH library logic YAML files from list to dict format. # Batch: HH # Per-file lines (before -> after): gfx950/Origami/Origami_nta4/gfx950_Cijk_Ailk_Bjlk_HF8_HF8S_BH_BiasSH_HAS_SAB_SCD_SAV_UserArgs.yaml: 40854 -> 29422 gfx950/Origami/Origami_nta4/gfx950_Cijk_Ailk_Bjlk_HF8_HHS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 40855 -> 29241 gfx950/Origami/Origami_nta4/gfx950_Cijk_Ailk_Bjlk_HF8_HSS_BH_BiasS_HAS_SAV_UserArgs.yaml: 40855 -> 29241 gfx950/Origami/Origami_nta4/gfx950_Cijk_Ailk_Bjlk_HH_B8F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 5238 -> 3932 gfx950/Origami/Origami_nta4/gfx950_Cijk_Ailk_Bjlk_HH_F8B8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 5238 -> 3932 gfx950/Origami/Origami_nta4/gfx950_Cijk_Ailk_Bjlk_HH_F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 5238 -> 3932 gfx950/Origami/Origami_nta4/gfx950_Cijk_Ailk_Bljk_HF8_HF8S_BH_BiasSH_HAS_SAB_SCD_SAV_UserArgs.yaml: 44214 -> 31726 gfx950/Origami/Origami_nta4/gfx950_Cijk_Ailk_Bljk_HF8_HHS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 44215 -> 31530 gfx950/Origami/Origami_nta4/gfx950_Cijk_Ailk_Bljk_HF8_HSS_BH_BiasS_HAS_SAV_UserArgs.yaml: 44215 -> 31530 gfx950/Origami/Origami_nta4/gfx950_Cijk_Ailk_Bljk_HH_B8F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 13974 -> 10145 gfx950/Origami/Origami_nta4/gfx950_Cijk_Ailk_Bljk_HH_F8B8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 13974 -> 10145 gfx950/Origami/Origami_nta4/gfx950_Cijk_Ailk_Bljk_HH_F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 13974 -> 10145 gfx950/Origami/Origami_nta4/gfx950_Cijk_Alik_Bjlk_HF8_HF8S_BH_BiasSH_HAS_SAB_SCD_SAV_UserArgs.yaml: 44662 -> 32040 gfx950/Origami/Origami_nta4/gfx950_Cijk_Alik_Bjlk_HF8_HHS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 44663 -> 31842 gfx950/Origami/Origami_nta4/gfx950_Cijk_Alik_Bjlk_HF8_HSS_BH_BiasS_HAS_SAV_UserArgs.yaml: 44663 -> 31842 gfx950/Origami/Origami_nta4/gfx950_Cijk_Alik_Bjlk_HH_B8F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 5014 -> 3773 gfx950/Origami/Origami_nta4/gfx950_Cijk_Alik_Bjlk_HH_F8B8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 5014 -> 3773 gfx950/Origami/Origami_nta4/gfx950_Cijk_Alik_Bjlk_HH_F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 5014 -> 3773 gfx950/Origami/Origami_nta4/gfx950_Cijk_Alik_Bljk_HF8_HF8S_BH_BiasSH_HAS_SAB_SCD_SAV_UserArgs.yaml: 48470 -> 34655 gfx950/Origami/Origami_nta4/gfx950_Cijk_Alik_Bljk_HF8_HHS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 48471 -> 34440 gfx950/Origami/Origami_nta4/gfx950_Cijk_Alik_Bljk_HF8_HSS_BH_BiasS_HAS_SAV_UserArgs.yaml: 48471 -> 34440 gfx950/Origami/Origami_nta4/gfx950_Cijk_Alik_Bljk_HH_B8F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 9046 -> 6632 gfx950/Origami/Origami_nta4/gfx950_Cijk_Alik_Bljk_HH_F8B8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 9046 -> 6632 gfx950/Origami/Origami_nta4/gfx950_Cijk_Alik_Bljk_HH_F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 9046 -> 6632 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Ailk_Bjlk_HF8_HF8S_BH_BiasSH_HAS_SAB_SCD_SAV_UserArgs.yaml: 40854 -> 29422 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Ailk_Bjlk_HF8_HHS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 40855 -> 29241 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Ailk_Bjlk_HF8_HSS_BH_BiasS_HAS_SAV_UserArgs.yaml: 40855 -> 29241 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Ailk_Bjlk_HH_B8F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 5238 -> 3932 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Ailk_Bjlk_HH_F8B8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 5238 -> 3932 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Ailk_Bjlk_HH_F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 5238 -> 3932 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Ailk_Bljk_HF8_HF8S_BH_BiasSH_HAS_SAB_SCD_SAV_UserArgs.yaml: 44214 -> 31726 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Ailk_Bljk_HF8_HHS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 44215 -> 31530 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Ailk_Bljk_HF8_HSS_BH_BiasS_HAS_SAV_UserArgs.yaml: 44215 -> 31530 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Ailk_Bljk_HH_B8F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 13974 -> 10145 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Ailk_Bljk_HH_F8B8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 13974 -> 10145 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Ailk_Bljk_HH_F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 13974 -> 10145 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Alik_Bjlk_HF8_HF8S_BH_BiasSH_HAS_SAB_SCD_SAV_UserArgs.yaml: 44662 -> 32040 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Alik_Bjlk_HF8_HHS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 44663 -> 31842 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Alik_Bjlk_HF8_HSS_BH_BiasS_HAS_SAV_UserArgs.yaml: 44663 -> 31842 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Alik_Bjlk_HH_B8F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 5014 -> 3773 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Alik_Bjlk_HH_F8B8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 5014 -> 3773 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Alik_Bjlk_HH_F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 5014 -> 3773 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Alik_Bljk_HF8_HF8S_BH_BiasSH_HAS_SAB_SCD_SAV_UserArgs.yaml: 48470 -> 34655 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Alik_Bljk_HF8_HHS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 48471 -> 34440 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Alik_Bljk_HF8_HSS_BH_BiasS_HAS_SAV_UserArgs.yaml: 48471 -> 34440 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Alik_Bljk_HH_B8F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 9046 -> 6632 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Alik_Bljk_HH_F8B8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 9046 -> 6632 gfx950/Origami/Origami_ntb4/gfx950_Cijk_Alik_Bljk_HH_F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 9046 -> 6632 gfx950/Origami/gfx950_Cijk_Ailk_Bjlk_HF8_HF8S_BH_BiasSH_HAS_SAB_SCD_SAV_UserArgs.yaml: 40854 -> 29240 gfx950/Origami/gfx950_Cijk_Ailk_Bjlk_HF8_HHS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 40855 -> 29059 gfx950/Origami/gfx950_Cijk_Ailk_Bjlk_HF8_HSS_BH_BiasS_HAS_SAV_UserArgs.yaml: 40855 -> 29059 gfx950/Origami/gfx950_Cijk_Ailk_Bjlk_HH_B8F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 5238 -> 3909 gfx950/Origami/gfx950_Cijk_Ailk_Bjlk_HH_F8B8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 5238 -> 3909 gfx950/Origami/gfx950_Cijk_Ailk_Bjlk_HH_F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 5238 -> 3909 gfx950/Origami/gfx950_Cijk_Ailk_Bljk_HF8_HF8S_BH_BiasSH_HAS_SAB_SCD_SAV_UserArgs.yaml: 44214 -> 31529 gfx950/Origami/gfx950_Cijk_Ailk_Bljk_HF8_HHS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 44215 -> 31333 gfx950/Origami/gfx950_Cijk_Ailk_Bljk_HF8_HSS_BH_BiasS_HAS_SAV_UserArgs.yaml: 44215 -> 31333 gfx950/Origami/gfx950_Cijk_Ailk_Bljk_HH_B8F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 13974 -> 10083 gfx950/Origami/gfx950_Cijk_Ailk_Bljk_HH_F8B8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 13974 -> 10083 gfx950/Origami/gfx950_Cijk_Ailk_Bljk_HH_F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 13974 -> 10083 gfx950/Origami/gfx950_Cijk_Alik_Bjlk_HF8_HF8S_BH_BiasSH_HAS_SAB_SCD_SAV_UserArgs.yaml: 44662 -> 31841 gfx950/Origami/gfx950_Cijk_Alik_Bjlk_HF8_HHS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 44663 -> 31643 gfx950/Origami/gfx950_Cijk_Alik_Bjlk_HF8_HSS_BH_BiasS_HAS_SAV_UserArgs.yaml: 44663 -> 31643 gfx950/Origami/gfx950_Cijk_Alik_Bjlk_HH_B8F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 5014 -> 3751 gfx950/Origami/gfx950_Cijk_Alik_Bjlk_HH_F8B8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 5014 -> 3751 gfx950/Origami/gfx950_Cijk_Alik_Bjlk_HH_F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 5014 -> 3751 gfx950/Origami/gfx950_Cijk_Alik_Bljk_HF8_HF8S_BH_BiasSH_HAS_SAB_SCD_SAV_UserArgs.yaml: 48470 -> 34439 gfx950/Origami/gfx950_Cijk_Alik_Bljk_HF8_HHS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 48471 -> 34224 gfx950/Origami/gfx950_Cijk_Alik_Bljk_HF8_HSS_BH_BiasS_HAS_SAV_UserArgs.yaml: 48471 -> 34224 gfx950/Origami/gfx950_Cijk_Alik_Bljk_HH_B8F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 9046 -> 6592 gfx950/Origami/gfx950_Cijk_Alik_Bljk_HH_F8B8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 9046 -> 6592 gfx950/Origami/gfx950_Cijk_Alik_Bljk_HH_F8HS_BH_BiasSH_HAS_SAV_UserArgs.yaml: 9046 -> 6592 Total: 1903272 -> 1363362 lines across 72 files.
✅ All Policy Checks Passed
📖 Need help? See the Policy FAQ for details on every check and how to fix failures. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
JIRA ID: AIHPBLAS-4216
Convert 72 gfx950 HH library logic YAML files from list to dict format.
Motivation
TensileLite is migrating gfx950 library logic YAML from the legacy list format to the canonical dict format. The dict format uses explicit root keys (
ArchitectureName,CUCount,ProblemType,Solutions, etc.) and is the formatTensileCreateLibrarynow emits by default.The HH datatype family continues the stacked gfx950 library logic conversion. Migrating these files improves maintainability, aligns shipped logic with the current serialization contract, and reduces YAML size without changing kernel selection behavior.
Technical Details
This PR converts 72 gfx950 HH library logic YAML files under:
projects/hipblaslt/library/src/amd_detail/rocblaslt/src/Tensile/Logic/asm_full/gfx950/gfx950/Origami/projects/hipblaslt/library/src/amd_detail/rocblaslt/src/Tensile/Logic/asm_full/gfx950/gfx950/Origami/Origami_nta4/projects/hipblaslt/library/src/amd_detail/rocblaslt/src/Tensile/Logic/asm_full/gfx950/gfx950/Origami/Origami_ntb4/Format changes (no semantic changes intended):
MinimumRequiredVersion,ScheduleName,ArchitectureName,CUCount,DeviceNames,ProblemType, …).ProblemTypefields are nested under aProblemType:mapping instead of inline list items.Solutions:dict keyed by solution name.True/False→true/false).Scope summary:
This PR is YAML-only; no runtime, generator, or test-code changes. It builds on
users/dsaffars/Lib_conv_gfx950_F4in the stacked PR series.Test Plan
For each of the 72 converted files, pre-conversion list-format YAML is compared against the new dict-format YAML using a script (same parser path as
TensileCreateLibrary):Test Result
For all 72/72 HH files parsed solutions, problem types, and matching tables are semantically identical between list and dict formats.
File integrity: No files added or removed; only format conversion within the existing 72 HH logic files (Origami, Origami/Origami_nta4, Origami/Origami_ntb4).
Line count: Total YAML size reduced from 1,903,272 to 1,363,362 lines (~28% reduction) with no intended change to kernel selection behavior.