AMDGPU: Don't add frame register to frame pseudos
authorMatt Arsenault <Matthew.Arsenault@amd.com>
Fri, 18 Sep 2020 20:04:36 +0000 (16:04 -0400)
committerMatt Arsenault <Matthew.Arsenault@amd.com>
Mon, 21 Sep 2020 20:18:47 +0000 (16:18 -0400)
We no longer treat the frame register like a function argument, so the
problem this avoided is no longer relevant.

llvm/lib/Target/AMDGPU/SIISelLowering.cpp

index a91a652..73c6564 100644 (file)
@@ -4179,13 +4179,8 @@ MachineBasicBlock *SITargetLowering::EmitInstrWithCustomInserter(
   case AMDGPU::ADJCALLSTACKDOWN: {
     const SIMachineFunctionInfo *Info = MF->getInfo<SIMachineFunctionInfo>();
     MachineInstrBuilder MIB(*MF, &MI);
-
-    // Add an implicit use of the frame offset reg to prevent the restore copy
-    // inserted after the call from being reorderd after stack operations in the
-    // the caller's frame.
     MIB.addReg(Info->getStackPtrOffsetReg(), RegState::ImplicitDefine)
-        .addReg(Info->getStackPtrOffsetReg(), RegState::Implicit)
-        .addReg(Info->getFrameOffsetReg(), RegState::Implicit);
+       .addReg(Info->getStackPtrOffsetReg(), RegState::Implicit);
     return BB;
   }
   case AMDGPU::SI_CALL_ISEL: {