[llvm-commits] CVS: llvm/lib/CodeGen/BranchFolding.cpp DwarfWriter.cpp ELFWriter.cpp LiveInterval.cpp LiveIntervalAnalysis.cpp LiveVariables.cpp

Reid Spencer reid at x10sys.com
Thu Nov 2 12:27:03 PST 2006



Changes in directory llvm/lib/CodeGen:

BranchFolding.cpp updated: 1.30 -> 1.31
DwarfWriter.cpp updated: 1.91 -> 1.92
ELFWriter.cpp updated: 1.26 -> 1.27
LiveInterval.cpp updated: 1.35 -> 1.36
LiveIntervalAnalysis.cpp updated: 1.187 -> 1.188
LiveVariables.cpp updated: 1.60 -> 1.61
---
Log message:

For PR786: http://llvm.org/PR786 :
Turn on -Wunused and -Wno-unused-parameter. Clean up most of the resulting
fall out by removing unused variables. Remaining warnings have to do with
unused functions (I didn't want to delete code without review) and unused
variables in generated code. Maintainers should clean up the remaining 
issues when they see them. All changes pass DejaGnu tests and Olden.


---
Diffs of the changes:  (+1 -7)

 BranchFolding.cpp        |    2 --
 DwarfWriter.cpp          |    1 -
 ELFWriter.cpp            |    1 -
 LiveInterval.cpp         |    1 -
 LiveIntervalAnalysis.cpp |    2 +-
 LiveVariables.cpp        |    1 -
 6 files changed, 1 insertion(+), 7 deletions(-)


Index: llvm/lib/CodeGen/BranchFolding.cpp
diff -u llvm/lib/CodeGen/BranchFolding.cpp:1.30 llvm/lib/CodeGen/BranchFolding.cpp:1.31
--- llvm/lib/CodeGen/BranchFolding.cpp:1.30	Wed Nov  1 13:36:29 2006
+++ llvm/lib/CodeGen/BranchFolding.cpp	Thu Nov  2 14:25:49 2006
@@ -766,7 +766,6 @@
         // Analyze the branch at the end of the pred.
         MachineBasicBlock *PredBB = *PI;
         MachineFunction::iterator PredFallthrough = PredBB; ++PredFallthrough;
-        MachineBasicBlock *PredTBB = 0, *PredFBB = 0;
         std::vector<MachineOperand> PredCond;
         if (PredBB != MBB && !CanFallThrough(PredBB)) {
           MBB->moveAfter(PredBB);
@@ -781,7 +780,6 @@
         // Analyze the branch at the end of the block before the succ.
         MachineBasicBlock *SuccBB = *SI;
         MachineFunction::iterator SuccPrev = SuccBB; --SuccPrev;
-        MachineBasicBlock *SuccPrevTBB = 0, *SuccPrevFBB = 0;
         std::vector<MachineOperand> SuccPrevCond;
         if (SuccBB != MBB && !CanFallThrough(SuccPrev)) {
           MBB->moveBefore(SuccBB);


Index: llvm/lib/CodeGen/DwarfWriter.cpp
diff -u llvm/lib/CodeGen/DwarfWriter.cpp:1.91 llvm/lib/CodeGen/DwarfWriter.cpp:1.92
--- llvm/lib/CodeGen/DwarfWriter.cpp:1.91	Thu Nov  2 14:12:39 2006
+++ llvm/lib/CodeGen/DwarfWriter.cpp	Thu Nov  2 14:25:49 2006
@@ -2132,7 +2132,6 @@
           } else {
             EmitInt8(DW_CFA_def_cfa);
             EOL("DW_CFA_def_cfa");
-            
             EmitULEB128Bytes(RI->getDwarfRegNum(Src.getRegister()));
             EOL("Register");
           }


Index: llvm/lib/CodeGen/ELFWriter.cpp
diff -u llvm/lib/CodeGen/ELFWriter.cpp:1.26 llvm/lib/CodeGen/ELFWriter.cpp:1.27
--- llvm/lib/CodeGen/ELFWriter.cpp:1.26	Fri May 12 01:33:48 2006
+++ llvm/lib/CodeGen/ELFWriter.cpp	Thu Nov  2 14:25:49 2006
@@ -113,7 +113,6 @@
 
   // Add padding zeros to the end of the buffer to make sure that the
   // function will start on the correct byte alignment within the section.
-  size_t SectionOff = OutBuffer->size();
   ELFWriter::align(*OutBuffer, Align);
 
   FnStart = OutBuffer->size();


Index: llvm/lib/CodeGen/LiveInterval.cpp
diff -u llvm/lib/CodeGen/LiveInterval.cpp:1.35 llvm/lib/CodeGen/LiveInterval.cpp:1.36
--- llvm/lib/CodeGen/LiveInterval.cpp:1.35	Sat Sep  2 00:26:59 2006
+++ llvm/lib/CodeGen/LiveInterval.cpp	Thu Nov  2 14:25:49 2006
@@ -289,7 +289,6 @@
   //
   // Also, if one range is a physreg and one is a vreg, we always merge from the
   // vreg into the physreg, which leaves the vreg intervals pristine.
-  unsigned OtherOffs = 1, ThisOffs = 0;
   if ((Other.ranges.size() > ranges.size() &&
       MRegisterInfo::isVirtualRegister(reg)) ||
       MRegisterInfo::isPhysicalRegister(Other.reg)) {


Index: llvm/lib/CodeGen/LiveIntervalAnalysis.cpp
diff -u llvm/lib/CodeGen/LiveIntervalAnalysis.cpp:1.187 llvm/lib/CodeGen/LiveIntervalAnalysis.cpp:1.188
--- llvm/lib/CodeGen/LiveIntervalAnalysis.cpp:1.187	Thu Sep 14 22:57:23 2006
+++ llvm/lib/CodeGen/LiveIntervalAnalysis.cpp	Thu Nov  2 14:25:49 2006
@@ -184,7 +184,7 @@
       if (tii_->isMoveInstr(*mii, srcReg, dstReg) &&
           (RegRep = rep(srcReg)) == rep(dstReg)) {
         // remove from def list
-        LiveInterval &interval = getOrCreateInterval(RegRep);
+        getOrCreateInterval(RegRep);
         RemoveMachineInstrFromMaps(mii);
         mii = mbbi->erase(mii);
         ++numPeep;


Index: llvm/lib/CodeGen/LiveVariables.cpp
diff -u llvm/lib/CodeGen/LiveVariables.cpp:1.60 llvm/lib/CodeGen/LiveVariables.cpp:1.61
--- llvm/lib/CodeGen/LiveVariables.cpp:1.60	Tue Oct  3 02:20:20 2006
+++ llvm/lib/CodeGen/LiveVariables.cpp	Thu Nov  2 14:25:49 2006
@@ -223,7 +223,6 @@
   for (df_ext_iterator<MachineBasicBlock*> DFI = df_ext_begin(Entry, Visited),
          E = df_ext_end(Entry, Visited); DFI != E; ++DFI) {
     MachineBasicBlock *MBB = *DFI;
-    unsigned BBNum = MBB->getNumber();
 
     // Loop over all of the instructions, processing them.
     for (MachineBasicBlock::iterator I = MBB->begin(), E = MBB->end();






More information about the llvm-commits mailing list