[PATCH] D63721: [AMDGPU] Remove unused variable AllSGPRSpilledToVGPRs. NFC

Bjorn Pettersson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 24 08:50:39 PDT 2019


This revision was automatically updated to reflect the committed changes.
Closed by commit rL364190: [AMDGPU] Remove unused variable AllSGPRSpilledToVGPRs. NFC (authored by bjope, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D63721?vs=206226&id=206230#toc

Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63721/new/

https://reviews.llvm.org/D63721

Files:
  llvm/trunk/lib/Target/AMDGPU/SIFrameLowering.cpp


Index: llvm/trunk/lib/Target/AMDGPU/SIFrameLowering.cpp
===================================================================
--- llvm/trunk/lib/Target/AMDGPU/SIFrameLowering.cpp
+++ llvm/trunk/lib/Target/AMDGPU/SIFrameLowering.cpp
@@ -754,11 +754,8 @@
   const SIInstrInfo *TII = ST.getInstrInfo();
   const SIRegisterInfo &TRI = TII->getRegisterInfo();
   SIMachineFunctionInfo *FuncInfo = MF.getInfo<SIMachineFunctionInfo>();
-  bool AllSGPRSpilledToVGPRs = false;
 
   if (TRI.spillSGPRToVGPR() && FuncInfo->hasSpilledSGPRs()) {
-    AllSGPRSpilledToVGPRs = true;
-
     // Process all SGPR spills before frame offsets are finalized. Ideally SGPRs
     // are spilled to VGPRs, in which case we can eliminate the stack usage.
     //
@@ -780,8 +777,7 @@
             bool Spilled = TRI.eliminateSGPRToVGPRSpillFrameIndex(MI, FI, RS);
             (void)Spilled;
             assert(Spilled && "failed to spill SGPR to VGPR when allocated");
-          } else
-            AllSGPRSpilledToVGPRs = false;
+          }
         }
       }
     }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D63721.206230.patch
Type: text/x-patch
Size: 1045 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190624/d2299328/attachment.bin>


More information about the llvm-commits mailing list