[PATCH] D11626: AMDGPU: Set SubRegIndex size and offset
Matt Arsenault
Matthew.Arsenault at amd.com
Wed Jul 29 23:30:35 PDT 2015
arsenm created this revision.
arsenm added a reviewer: tstellarAMD.
arsenm added a subscriber: llvm-commits.
I'm not sure what reasons the comment here could have had for not setting these. Without these set, there is an assertion hit during DWARF emission.
http://reviews.llvm.org/D11626
Files:
lib/Target/AMDGPU/AMDGPURegisterInfo.td
Index: lib/Target/AMDGPU/AMDGPURegisterInfo.td
===================================================================
--- lib/Target/AMDGPU/AMDGPURegisterInfo.td
+++ lib/Target/AMDGPU/AMDGPURegisterInfo.td
@@ -14,8 +14,7 @@
let Namespace = "AMDGPU" in {
foreach Index = 0-15 in {
- // Indices are used in a variety of ways here, so don't set a size/offset.
- def sub#Index : SubRegIndex<-1, -1>;
+ def sub#Index : SubRegIndex<32, !shl(Index, 5)>;
}
def INDIRECT_BASE_ADDR : Register <"INDIRECT_BASE_ADDR">;
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D11626.30995.patch
Type: text/x-patch
Size: 515 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150730/b5968449/attachment.bin>
More information about the llvm-commits
mailing list