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();



_______________________________________________
llvm-commits mailing list
llvm-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

Reply via email to