Also makes another step towards resolving
https://github.com/llvm/llvm-project/issues/38652
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
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.
Comment Actions
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.