[llvm-commits] CVS: llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp PPCCodeEmitter.cpp PPCISelDAGToDAG.cpp PPCISelLowering.cpp PPCMachOWriter.cpp PPCRegisterInfo.cpp

[Reid Spencer]

Changes in directory llvm/lib/Target/PowerPC:

PPCAsmPrinter.cpp updated: 1.205 -> 1.206
PPCCodeEmitter.cpp updated: 1.67 -> 1.68
PPCISelDAGToDAG.cpp updated: 1.212 -> 1.213
PPCISelLowering.cpp updated: 1.217 -> 1.218
PPCMachOWriter.cpp updated: 1.5 -> 1.6
PPCRegisterInfo.cpp updated: 1.76 -> 1.77
---
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 -9)

 PPCAsmPrinter.cpp   |    2 --
 PPCCodeEmitter.cpp  |    1 -
 PPCISelDAGToDAG.cpp |    2 --
 PPCISelLowering.cpp |    2 --
 PPCMachOWriter.cpp  |    2 +-
 PPCRegisterInfo.cpp |    1 -
 6 files changed, 1 insertion(+), 9 deletions(-)


Index: llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp
diff -u llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp:1.205 llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp:1.206
--- llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp:1.205	Tue Oct 31 02:31:24 2006
+++ llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp	Thu Nov  2 14:25:49 2006
@@ -249,7 +249,6 @@
     DarwinAsmPrinter(std::ostream &O, PPCTargetMachine &TM,
                      const TargetAsmInfo *T)
       : PPCAsmPrinter(O, TM, T), DW(O, this, T) {
-      bool isPPC64 = Subtarget.isPPC64();
     }
 
     virtual const char *getPassName() const {
@@ -308,7 +307,6 @@
     // Computing the address of a global symbol, not calling it.
     GlobalValue *GV = MO.getGlobal();
     std::string Name = Mang->getValueName(GV);
-    int offset = MO.getOffset();
 
     // External or weakly linked global variables need non-lazily-resolved stubs
     if (TM.getRelocationModel() != Reloc::Static) {


Index: llvm/lib/Target/PowerPC/PPCCodeEmitter.cpp
diff -u llvm/lib/Target/PowerPC/PPCCodeEmitter.cpp:1.67 llvm/lib/Target/PowerPC/PPCCodeEmitter.cpp:1.68
--- llvm/lib/Target/PowerPC/PPCCodeEmitter.cpp:1.67	Sun Aug 27 07:54:01 2006
+++ llvm/lib/Target/PowerPC/PPCCodeEmitter.cpp	Thu Nov  2 14:25:49 2006
@@ -91,7 +91,6 @@
   
   for (MachineBasicBlock::iterator I = MBB.begin(), E = MBB.end(); I != E; ++I){
     MachineInstr &MI = *I;
-    unsigned Opcode = MI.getOpcode();
     switch (MI.getOpcode()) {
     default:
       MCE.emitWordBE(getBinaryCodeForInstr(*I));


Index: llvm/lib/Target/PowerPC/PPCISelDAGToDAG.cpp
diff -u llvm/lib/Target/PowerPC/PPCISelDAGToDAG.cpp:1.212 llvm/lib/Target/PowerPC/PPCISelDAGToDAG.cpp:1.213
--- llvm/lib/Target/PowerPC/PPCISelDAGToDAG.cpp:1.212	Mon Oct 30 17:02:25 2006
+++ llvm/lib/Target/PowerPC/PPCISelDAGToDAG.cpp	Thu Nov  2 14:25:49 2006
@@ -1134,7 +1134,6 @@
     SDOperand CCReg = SelectCC(N->getOperand(0), N->getOperand(1), CC);
     unsigned BROpc = getBCCForSetCC(CC);
 
-    bool isFP = MVT::isFloatingPoint(N->getValueType(0));
     unsigned SelectCCOp;
     if (N->getValueType(0) == MVT::i32)
       SelectCCOp = PPC::SELECT_CC_I4;
@@ -1218,7 +1217,6 @@
   SDOperand Chain(0, 0);
   SDOperand N1(0, 0);
   SDOperand Tmp0(0, 0);
-  SDNode *ResNode;
   Chain = N.getOperand(0);
   N1 = N.getOperand(1);
   


Index: llvm/lib/Target/PowerPC/PPCISelLowering.cpp
diff -u llvm/lib/Target/PowerPC/PPCISelLowering.cpp:1.217 llvm/lib/Target/PowerPC/PPCISelLowering.cpp:1.218
--- llvm/lib/Target/PowerPC/PPCISelLowering.cpp:1.217	Wed Nov  1 19:44:04 2006
+++ llvm/lib/Target/PowerPC/PPCISelLowering.cpp	Thu Nov  2 14:25:49 2006
@@ -942,9 +942,7 @@
 
 static SDOperand LowerCALL(SDOperand Op, SelectionDAG &DAG) {
   SDOperand Chain = Op.getOperand(0);
-  unsigned CallingConv= cast<ConstantSDNode>(Op.getOperand(1))->getValue();
   bool isVarArg       = cast<ConstantSDNode>(Op.getOperand(2))->getValue() != 0;
-  bool isTailCall     = cast<ConstantSDNode>(Op.getOperand(3))->getValue() != 0;
   SDOperand Callee    = Op.getOperand(4);
   unsigned NumOps     = (Op.getNumOperands() - 5) / 2;
 


Index: llvm/lib/Target/PowerPC/PPCMachOWriter.cpp
diff -u llvm/lib/Target/PowerPC/PPCMachOWriter.cpp:1.5 llvm/lib/Target/PowerPC/PPCMachOWriter.cpp:1.6
--- llvm/lib/Target/PowerPC/PPCMachOWriter.cpp:1.5	Sun Sep 10 18:03:44 2006
+++ llvm/lib/Target/PowerPC/PPCMachOWriter.cpp	Thu Nov  2 14:25:49 2006
@@ -99,7 +99,7 @@
   case PPC::reloc_pcrel_bx:
     Addr -= MR.getMachineCodeOffset();
     Addr >>= 2;
-    Addr & 0xFFFFFF;
+    // ??? Addr & 0xFFFFFF;  Possibly this was supposed to be Addr &= 0xFFFFFF ?
     Addr <<= 2;
     Addr |= (MOS.SectionData[MR.getMachineCodeOffset()] << 24);
     fixword(MOS.SectionData, Addr, MR.getMachineCodeOffset());


Index: llvm/lib/Target/PowerPC/PPCRegisterInfo.cpp
diff -u llvm/lib/Target/PowerPC/PPCRegisterInfo.cpp:1.76 llvm/lib/Target/PowerPC/PPCRegisterInfo.cpp:1.77
--- llvm/lib/Target/PowerPC/PPCRegisterInfo.cpp:1.76	Mon Sep  4 21:31:13 2006
+++ llvm/lib/Target/PowerPC/PPCRegisterInfo.cpp	Thu Nov  2 14:25:49 2006
@@ -370,7 +370,6 @@
 //
 static bool hasFP(const MachineFunction &MF) {
   const MachineFrameInfo *MFI = MF.getFrameInfo();
-  unsigned TargetAlign = MF.getTarget().getFrameInfo()->getStackAlignment();
 
   // If frame pointers are forced, or if there are variable sized stack objects,
   // use a frame pointer.