[llvm] r312844 - RegAllocFast: Remove write-only set; NFC

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 8 17:52:42 PDT 2017


Author: matze
Date: Fri Sep  8 17:52:42 2017
New Revision: 312844

URL: http://llvm.org/viewvc/llvm-project?rev=312844&view=rev
Log:
RegAllocFast: Remove write-only set; NFC

Modified:
    llvm/trunk/lib/CodeGen/RegAllocFast.cpp

Modified: llvm/trunk/lib/CodeGen/RegAllocFast.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/RegAllocFast.cpp?rev=312844&r1=312843&r2=312844&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/RegAllocFast.cpp (original)
+++ llvm/trunk/lib/CodeGen/RegAllocFast.cpp Fri Sep  8 17:52:42 2017
@@ -133,11 +133,6 @@ namespace {
       return false;
     }
 
-    // SkippedInstrs - Descriptors of instructions whose clobber list was
-    // ignored because all registers were spilled. It is still necessary to
-    // mark all the clobbered registers as used by the function.
-    SmallPtrSet<const MCInstrDesc*, 4> SkippedInstrs;
-
     // isBulkSpilling - This flag is set when LiveRegMap will be cleared
     // completely after spilling all live registers. LiveRegMap entries should
     // not be erased.
@@ -1025,10 +1020,6 @@ void RAFast::AllocateBasicBlock() {
       // those for virtual registers in between.
       DEBUG(dbgs() << "  Spilling remaining registers before call.\n");
       spillAll(MI);
-
-      // The imp-defs are skipped below, but we still need to mark those
-      // registers as used by the function.
-      SkippedInstrs.insert(&MCID);
     }
 
     // Third scan.
@@ -1112,7 +1103,6 @@ bool RAFast::runOnMachineFunction(Machin
   // replaced. Remove the virtual registers.
   MRI->clearVirtRegs();
 
-  SkippedInstrs.clear();
   StackSlotForVirtReg.clear();
   LiveDbgValueMap.clear();
   return true;




More information about the llvm-commits mailing list