[llvm-commits] [llvm] r91764 - in /llvm/trunk: include/llvm/MC/MCAssembler.h lib/CodeGen/AsmPrinter/AsmPrinter.cpp lib/CodeGen/ELF.h lib/CodeGen/PBQP/GraphBase.h lib/CodeGen/PBQP/HeuristicSolver.h lib/CodeGen/RegAllocLinearScan.cpp lib/CodeGen/SimpleRegisterCoalescing.h lib/Target/ARM/ARMLoadStoreOptimizer.cpp lib/Target/Mips/MipsSubtarget.h lib/Transforms/Hello/Hello.cpp

Douglas Gregor dgregor at apple.com
Fri Dec 18 23:05:23 PST 2009


Author: dgregor
Date: Sat Dec 19 01:05:23 2009
New Revision: 91764

URL: http://llvm.org/viewvc/llvm-project?rev=91764&view=rev
Log:
Fix a bunch of little errors that Clang complains about when its being pedantic

Modified:
    llvm/trunk/include/llvm/MC/MCAssembler.h
    llvm/trunk/lib/CodeGen/AsmPrinter/AsmPrinter.cpp
    llvm/trunk/lib/CodeGen/ELF.h
    llvm/trunk/lib/CodeGen/PBQP/GraphBase.h
    llvm/trunk/lib/CodeGen/PBQP/HeuristicSolver.h
    llvm/trunk/lib/CodeGen/RegAllocLinearScan.cpp
    llvm/trunk/lib/CodeGen/SimpleRegisterCoalescing.h
    llvm/trunk/lib/Target/ARM/ARMLoadStoreOptimizer.cpp
    llvm/trunk/lib/Target/Mips/MipsSubtarget.h
    llvm/trunk/lib/Transforms/Hello/Hello.cpp

Modified: llvm/trunk/include/llvm/MC/MCAssembler.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/MC/MCAssembler.h?rev=91764&r1=91763&r2=91764&view=diff

==============================================================================
--- llvm/trunk/include/llvm/MC/MCAssembler.h (original)
+++ llvm/trunk/include/llvm/MC/MCAssembler.h Sat Dec 19 01:05:23 2009
@@ -76,7 +76,7 @@
   virtual uint64_t getMaxFileSize() const {
     assert(0 && "Invalid getMaxFileSize call!");
     return 0;
-  };
+  }
 
   /// @name Assembler Backend Support
   /// @{

Modified: llvm/trunk/lib/CodeGen/AsmPrinter/AsmPrinter.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/AsmPrinter/AsmPrinter.cpp?rev=91764&r1=91763&r2=91764&view=diff

==============================================================================
--- llvm/trunk/lib/CodeGen/AsmPrinter/AsmPrinter.cpp (original)
+++ llvm/trunk/lib/CodeGen/AsmPrinter/AsmPrinter.cpp Sat Dec 19 01:05:23 2009
@@ -236,7 +236,7 @@
     const MCSection *S;
     unsigned Alignment;
     SmallVector<unsigned, 4> CPEs;
-    SectionCPs(const MCSection *s, unsigned a) : S(s), Alignment(a) {};
+    SectionCPs(const MCSection *s, unsigned a) : S(s), Alignment(a) {}
   };
 }
 

Modified: llvm/trunk/lib/CodeGen/ELF.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/ELF.h?rev=91764&r1=91763&r2=91764&view=diff

==============================================================================
--- llvm/trunk/lib/CodeGen/ELF.h (original)
+++ llvm/trunk/lib/CodeGen/ELF.h Sat Dec 19 01:05:23 2009
@@ -82,14 +82,14 @@
     const GlobalValue *getGlobalValue() const {
       assert(SourceType == isGV && "This is not a global value");
       return Source.GV;
-    };
+    }
 
     // getExternalSym - If this is an external symbol which originated the
     // elf symbol, return a reference to it.
     const char *getExternalSymbol() const {
       assert(SourceType == isExtSym && "This is not an external symbol");
       return Source.Ext;
-    };
+    }
 
     // getGV - From a global value return a elf symbol to represent it
     static ELFSym *getGV(const GlobalValue *GV, unsigned Bind,

Modified: llvm/trunk/lib/CodeGen/PBQP/GraphBase.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/PBQP/GraphBase.h?rev=91764&r1=91763&r2=91764&view=diff

==============================================================================
--- llvm/trunk/lib/CodeGen/PBQP/GraphBase.h (original)
+++ llvm/trunk/lib/CodeGen/PBQP/GraphBase.h Sat Dec 19 01:05:23 2009
@@ -298,7 +298,7 @@
 
     for (ConstAdjEdgeIterator adjEdgeItr = adjEdgesBegin(node1Itr),
          adjEdgeEnd = adjEdgesEnd(node1Itr);
-         adjEdgeItr != adjEdgesEnd; ++adjEdgeItr) {
+         adjEdgeItr != adjEdgeEnd; ++adjEdgeItr) {
       if ((getEdgeNode1Itr(*adjEdgeItr) == node2Itr) ||
           (getEdgeNode2Itr(*adjEdgeItr) == node2Itr)) {
         return *adjEdgeItr;

Modified: llvm/trunk/lib/CodeGen/PBQP/HeuristicSolver.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/PBQP/HeuristicSolver.h?rev=91764&r1=91763&r2=91764&view=diff

==============================================================================
--- llvm/trunk/lib/CodeGen/PBQP/HeuristicSolver.h (original)
+++ llvm/trunk/lib/CodeGen/PBQP/HeuristicSolver.h Sat Dec 19 01:05:23 2009
@@ -536,7 +536,7 @@
       else reductionFinished = true;
     }
       
-  };
+  }
 
   void processR1() {
 

Modified: llvm/trunk/lib/CodeGen/RegAllocLinearScan.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/RegAllocLinearScan.cpp?rev=91764&r1=91763&r2=91764&view=diff

==============================================================================
--- llvm/trunk/lib/CodeGen/RegAllocLinearScan.cpp (original)
+++ llvm/trunk/lib/CodeGen/RegAllocLinearScan.cpp Sat Dec 19 01:05:23 2009
@@ -891,7 +891,7 @@
     const RALinScan &Allocator;
 
   public:
-    WeightCompare(const RALinScan &Alloc) : Allocator(Alloc) {};
+    WeightCompare(const RALinScan &Alloc) : Allocator(Alloc) {}
 
     typedef std::pair<unsigned, float> RegWeightPair;
     bool operator()(const RegWeightPair &LHS, const RegWeightPair &RHS) const {

Modified: llvm/trunk/lib/CodeGen/SimpleRegisterCoalescing.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SimpleRegisterCoalescing.h?rev=91764&r1=91763&r2=91764&view=diff

==============================================================================
--- llvm/trunk/lib/CodeGen/SimpleRegisterCoalescing.h (original)
+++ llvm/trunk/lib/CodeGen/SimpleRegisterCoalescing.h Sat Dec 19 01:05:23 2009
@@ -33,7 +33,7 @@
     MachineInstr *MI;
     unsigned LoopDepth;
     CopyRec(MachineInstr *mi, unsigned depth)
-      : MI(mi), LoopDepth(depth) {};
+      : MI(mi), LoopDepth(depth) {}
   };
 
   class SimpleRegisterCoalescing : public MachineFunctionPass,
@@ -85,7 +85,7 @@
     bool coalesceFunction(MachineFunction &mf, RegallocQuery &) {
       // This runs as an independent pass, so don't do anything.
       return false;
-    };
+    }
 
     /// print - Implement the dump method.
     virtual void print(raw_ostream &O, const Module* = 0) const;

Modified: llvm/trunk/lib/Target/ARM/ARMLoadStoreOptimizer.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/ARMLoadStoreOptimizer.cpp?rev=91764&r1=91763&r2=91764&view=diff

==============================================================================
--- llvm/trunk/lib/Target/ARM/ARMLoadStoreOptimizer.cpp (original)
+++ llvm/trunk/lib/Target/ARM/ARMLoadStoreOptimizer.cpp Sat Dec 19 01:05:23 2009
@@ -78,7 +78,7 @@
       MachineBasicBlock::iterator MBBI;
       bool Merged;
       MemOpQueueEntry(int o, int p, MachineBasicBlock::iterator i)
-        : Offset(o), Position(p), MBBI(i), Merged(false) {};
+        : Offset(o), Position(p), MBBI(i), Merged(false) {}
     };
     typedef SmallVector<MemOpQueueEntry,8> MemOpQueue;
     typedef MemOpQueue::iterator MemOpQueueIter;

Modified: llvm/trunk/lib/Target/Mips/MipsSubtarget.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MipsSubtarget.h?rev=91764&r1=91763&r2=91764&view=diff

==============================================================================
--- llvm/trunk/lib/Target/Mips/MipsSubtarget.h (original)
+++ llvm/trunk/lib/Target/Mips/MipsSubtarget.h Sat Dec 19 01:05:23 2009
@@ -102,21 +102,21 @@
   bool isMips1() const { return MipsArchVersion == Mips1; }
 
   bool isLittle() const { return IsLittle; }
-  bool isFP64bit() const { return IsFP64bit; };
-  bool isGP64bit() const { return IsGP64bit; };
-  bool isGP32bit() const { return !IsGP64bit; };
-  bool isSingleFloat() const { return IsSingleFloat; };
-  bool isNotSingleFloat() const { return !IsSingleFloat; };
-  bool hasVFPU() const { return HasVFPU; };
-  bool isLinux() const { return IsLinux; };
+  bool isFP64bit() const { return IsFP64bit; }
+  bool isGP64bit() const { return IsGP64bit; }
+  bool isGP32bit() const { return !IsGP64bit; }
+  bool isSingleFloat() const { return IsSingleFloat; }
+  bool isNotSingleFloat() const { return !IsSingleFloat; }
+  bool hasVFPU() const { return HasVFPU; }
+  bool isLinux() const { return IsLinux; }
 
   /// Features related to the presence of specific instructions.
-  bool hasSEInReg()   const { return HasSEInReg; };
-  bool hasCondMov()   const { return HasCondMov; };
-  bool hasMulDivAdd() const { return HasMulDivAdd; };
-  bool hasMinMax()    const { return HasMinMax; };
-  bool hasSwap()      const { return HasSwap; };
-  bool hasBitCount()  const { return HasBitCount; };
+  bool hasSEInReg()   const { return HasSEInReg; }
+  bool hasCondMov()   const { return HasCondMov; }
+  bool hasMulDivAdd() const { return HasMulDivAdd; }
+  bool hasMinMax()    const { return HasMinMax; }
+  bool hasSwap()      const { return HasSwap; }
+  bool hasBitCount()  const { return HasBitCount; }
 };
 } // End llvm namespace
 

Modified: llvm/trunk/lib/Transforms/Hello/Hello.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Hello/Hello.cpp?rev=91764&r1=91763&r2=91764&view=diff

==============================================================================
--- llvm/trunk/lib/Transforms/Hello/Hello.cpp (original)
+++ llvm/trunk/lib/Transforms/Hello/Hello.cpp Sat Dec 19 01:05:23 2009
@@ -56,7 +56,7 @@
     // We don't modify the program, so we preserve all analyses
     virtual void getAnalysisUsage(AnalysisUsage &AU) const {
       AU.setPreservesAll();
-    };
+    }
   };
 }
 





More information about the llvm-commits mailing list