]> granicus.if.org Git - llvm/commitdiff
AMDGPU: Don't add same implicit use multiple times
authorMatt Arsenault <Matthew.Arsenault@amd.com>
Mon, 12 Jun 2017 17:19:20 +0000 (17:19 +0000)
committerMatt Arsenault <Matthew.Arsenault@amd.com>
Mon, 12 Jun 2017 17:19:20 +0000 (17:19 +0000)
For the last component, the same register use
was added as an implicit use and another implicit kill use.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@305205 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/AMDGPU/SIInstrInfo.cpp

index d206a5ec9d5386f0a20cf8a172db747bf99a621d..1097814e99ce22024e59af99e4969f8b562901b3 100644 (file)
@@ -468,13 +468,11 @@ void SIInstrInfo::copyPhysReg(MachineBasicBlock &MBB,
 
     Builder.addReg(RI.getSubReg(SrcReg, SubIdx));
 
-    if (Idx == SubIndices.size() - 1)
-      Builder.addReg(SrcReg, getKillRegState(KillSrc) | RegState::Implicit);
-
     if (Idx == 0)
       Builder.addReg(DestReg, RegState::Define | RegState::Implicit);
 
-    Builder.addReg(SrcReg, RegState::Implicit);
+    bool UseKill = KillSrc && Idx == SubIndices.size() - 1;
+    Builder.addReg(SrcReg, getKillRegState(UseKill) | RegState::Implicit);
   }
 }