[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