This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU][GFX10] Support base+soffset+offset SMEM stores.
ClosedPublic

Authored by kosarev on May 11 2022, 6:08 AM.

Diff Detail

Event Timeline

kosarev created this revision.May 11 2022, 6:08 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 11 2022, 6:08 AM
kosarev requested review of this revision.May 11 2022, 6:08 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 11 2022, 6:08 AM

A couple notes:

  • It seems I missed tests for scratch loads in D125117. Will add them later.
  • Looking at how test/MC/AMDGPU/smem.s is organised, it's not perfectly clear to me if we want offset-validation tests for the stores where we already have them for loads.
foad accepted this revision.May 11 2022, 6:31 AM

LGTM, thanks!

This revision is now accepted and ready to land.May 11 2022, 6:31 AM
dp accepted this revision.May 11 2022, 11:59 AM

LGTM

Looking at how test/MC/AMDGPU/smem.s is organised, it's not perfectly clear to me if we want offset-validation tests for the stores where we already have them for loads.

AFAIK, all features should be covered by lit tests, but there are no specific requirements on test coverage. If it is clear that existing tests cover new functionality, you do not have to add new tests.

This revision was landed with ongoing or failed builds.May 12 2022, 12:49 AM
This revision was automatically updated to reflect the committed changes.