[PATCH] D31872: AMDGPU: Minor SReg64 register class refactoring
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 10 15:27:07 PDT 2017
arsenm added a comment.
In https://reviews.llvm.org/D31872#723085, @kzhuravl wrote:
> In https://reviews.llvm.org/D31872#722575, @arsenm wrote:
>
> > I think this requires changing the register class for SSrc_32/VSrc_32 to be the one that includes the special 32-bit regs. I think I started doing this to start supporting vccz and the other special 1-bit inputs
>
>
> I do not think so. src_*_base and src_*_limit are 64 bit inline constants. Here is an example of src_shared_base:
>
> {SMB.shared_base[15:0], 48’h000000000000}
>
>
> src_*_base and src_*_limit also do not have subregisters.
According to the manual it says if used with a 32-bit input, the 32 LSBs are used, so I guess it has sub0 but not sub1
https://reviews.llvm.org/D31872
More information about the llvm-commits
mailing list