GFX9 subtargets from GFX90A onwards lack GDS but still have GWS.
Details
Details
- Reviewers
b-sumner arsenm - Group Reviewers
Restricted Project - Commits
- rGc2093b85044d: [AMDGPU] Add target features for GDS and GWS
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
We do need a feature that include GWS. GDS was dropped before GWS in GFX9, but compute never used GDS. So GDSandGWS would probably work.
Comment Actions
What about gfx909 and gfx90c?
In the current patch I have assumed gfx908 and gfx909 both have GDS, and gfx90a and gfx90c (and gfx94x) lack it.
Comment Actions
Somewhat confusingly, gfx90c is an older APU (Renoir IIRC), and not a successor to gfx90a, which would still have GDS, I believe.
llvm/test/CodeGen/AMDGPU/gds-unsupported.ll | ||
---|---|---|
1 | don't need -verify-machineinstrs |
don't need -verify-machineinstrs