]> granicus.if.org Git - llvm/commitdiff
AMDGPU: Fix crash with undef vreg input operand
authorMatt Arsenault <Matthew.Arsenault@amd.com>
Tue, 20 Jun 2017 18:28:02 +0000 (18:28 +0000)
committerMatt Arsenault <Matthew.Arsenault@amd.com>
Tue, 20 Jun 2017 18:28:02 +0000 (18:28 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@305814 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/AMDGPU/SIFoldOperands.cpp
test/CodeGen/AMDGPU/constant-fold-imm-immreg.mir

index e10f1ed3762e84b345b562904c9e5fcd129766db..bf6b7c55a69ad6c54e32c510d1c30c47610070a7 100644 (file)
@@ -470,7 +470,7 @@ static MachineOperand *getImmOrMaterializedImm(MachineRegisterInfo &MRI,
       return &Op;
 
     MachineInstr *Def = MRI.getVRegDef(Op.getReg());
-    if (Def->isMoveImmediate()) {
+    if (Def && Def->isMoveImmediate()) {
       MachineOperand &ImmSrc = Def->getOperand(1);
       if (ImmSrc.isImm())
         return &ImmSrc;
index 62b47beb125186c2eeb9cfdc3df218f2c4579845..4d8e608c947846f6d79509756a593528c0363ba8 100644 (file)
     ret void
   }
 
+  define amdgpu_kernel void @undefined_vreg_operand() {
+    unreachable
+  }
+
   declare i32 @llvm.amdgcn.workitem.id.x() #1
 
   attributes #0 = { nounwind }
@@ -856,3 +860,20 @@ body:             |
     S_ENDPGM
 
 ...
+---
+# There is only an undef use operand for %1, so there is no
+# corresponding defining instruction
+
+name: undefined_vreg_operand
+tracksRegLiveness: true
+registers:
+  - { id: 0, class: vgpr_32, preferred-register: '' }
+  - { id: 1, class: vgpr_32, preferred-register: '' }
+  - { id: 2, class: vgpr_32, preferred-register: '' }
+body:             |
+  bb.0:
+    %0 = V_MOV_B32_e32 0, implicit %exec
+    %2 = V_XOR_B32_e64 killed %0, undef %1, implicit %exec
+    S_ENDPGM
+
+...