[llvm] [AMDGPU][True16][CodeGen] atomic load/store i8 in true16 mode (PR #143044)

Brox Chen via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 9 12:04:33 PDT 2025


broxigarchen wrote:

> > Do we have any tests that the hi/lo half that is unused by the instruction is reused by another instruction? I think for the non-atomic test cases we do, and these should work the same, but the atomic tests just don't cover that behavior.
> 
> Let me add a test

I took a look and it seems some of the d16_hi insts could not be selected by atomic load due to some other codegen patterns impact. But I will merge this patch fiirst and open another patch later, since this patch applies vgpr16 which is a functional issue, while the codegen patterns are a code quality change.

https://github.com/llvm/llvm-project/pull/143044


More information about the llvm-commits mailing list