[lld] r243060 - Fix -Wextra-semi.

[Rui Ueyama]

Author: ruiu
Date: Thu Jul 23 18:03:55 2015
New Revision: 243060

URL: http://llvm.org/viewvc/llvm-project?rev=243060&view=rev
Log:
Fix -Wextra-semi.

Patch from Eugene.Zelenko!

Modified:
    lld/trunk/include/lld/Core/LinkingContext.h
    lld/trunk/include/lld/ReaderWriter/ELFLinkingContext.h
    lld/trunk/lib/ReaderWriter/PECOFF/IdataPass.h
    lld/trunk/lib/ReaderWriter/PECOFF/LinkerGeneratedSymbolFile.h
    lld/trunk/unittests/DriverTests/GnuLdDriverTest.cpp

Modified: lld/trunk/include/lld/Core/LinkingContext.h
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/include/lld/Core/LinkingContext.h?rev=243060&r1=243059&r2=243060&view=diff
==============================================================================
--- lld/trunk/include/lld/Core/LinkingContext.h (original)
+++ lld/trunk/include/lld/Core/LinkingContext.h Thu Jul 23 18:03:55 2015
@@ -61,7 +61,7 @@ public:
   /// should be marked live (along with all Atoms they reference).  Usually
   /// this method returns false for main executables, but true for dynamic
   /// shared libraries.
-  bool globalsAreDeadStripRoots() const { return _globalsAreDeadStripRoots; };
+  bool globalsAreDeadStripRoots() const { return _globalsAreDeadStripRoots; }
 
   /// Only used if deadStrip() returns true.  This method returns the names
   /// of DefinedAtoms that should be marked live (along with all Atoms they

Modified: lld/trunk/include/lld/ReaderWriter/ELFLinkingContext.h
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/include/lld/ReaderWriter/ELFLinkingContext.h?rev=243060&r1=243059&r2=243060&view=diff
==============================================================================
--- lld/trunk/include/lld/ReaderWriter/ELFLinkingContext.h (original)
+++ lld/trunk/include/lld/ReaderWriter/ELFLinkingContext.h Thu Jul 23 18:03:55 2015
@@ -282,7 +282,7 @@ public:
   void addSearchPath(StringRef ref) { _inputSearchPaths.push_back(ref); }
 
   // Retrieve search path list.
-  StringRefVector getSearchPaths() { return _inputSearchPaths; };
+  StringRefVector getSearchPaths() { return _inputSearchPaths; }
 
   // By default, the linker would merge sections that are read only with
   // segments that have read and execute permissions. When the user specifies a
@@ -336,8 +336,8 @@ public:
   void addWrapForSymbol(StringRef sym) { _wrapCalls.insert(sym); }
 
   // \brief Set DT_FLAGS flag.
-  void setDTFlag(DTFlag f) { _dtFlags |= f; };
-  bool getDTFlag(DTFlag f) { return (_dtFlags & f); };
+  void setDTFlag(DTFlag f) { _dtFlags |= f; }
+  bool getDTFlag(DTFlag f) { return (_dtFlags & f); }
 
   const llvm::StringSet<> &wrapCalls() const { return _wrapCalls; }
 

Modified: lld/trunk/lib/ReaderWriter/PECOFF/IdataPass.h
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/PECOFF/IdataPass.h?rev=243060&r1=243059&r2=243060&view=diff
==============================================================================
--- lld/trunk/lib/ReaderWriter/PECOFF/IdataPass.h (original)
+++ lld/trunk/lib/ReaderWriter/PECOFF/IdataPass.h Thu Jul 23 18:03:55 2015
@@ -87,7 +87,7 @@ public:
 
   StringRef customSectionName() const override {
     return _sectionName;
-  };
+  }
 
 private:
   std::vector<uint8_t> assembleRawContent(uint64_t contents, bool is64);

Modified: lld/trunk/lib/ReaderWriter/PECOFF/LinkerGeneratedSymbolFile.h
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/PECOFF/LinkerGeneratedSymbolFile.h?rev=243060&r1=243059&r2=243060&view=diff
==============================================================================
--- lld/trunk/lib/ReaderWriter/PECOFF/LinkerGeneratedSymbolFile.h (original)
+++ lld/trunk/lib/ReaderWriter/PECOFF/LinkerGeneratedSymbolFile.h Thu Jul 23 18:03:55 2015
@@ -63,7 +63,7 @@ public:
     _defined.addReference(std::unique_ptr<SimpleReference>(ref));
     addAtom(_defined);
     addAtom(_undefined);
-  };
+  }
 
 private:
   SimpleUndefinedAtom _undefined;
@@ -79,7 +79,7 @@ public:
       : SimpleFile("<symbol-rename>"), _fromSym(from), _toSym(to),
         _from(*this, _fromSym, &_to), _to(*this, _toSym) {
     addAtom(_from);
-  };
+  }
 
 private:
   std::string _fromSym;
@@ -100,7 +100,7 @@ public:
         _imageBaseAtom(*this, ctx.decorateSymbol("__ImageBase"),
                        Atom::scopeGlobal, ctx.getBaseAddress()) {
     addAtom(_imageBaseAtom);
-  };
+  }
 
 private:
   SimpleAbsoluteAtom _imageBaseAtom;

Modified: lld/trunk/unittests/DriverTests/GnuLdDriverTest.cpp
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/unittests/DriverTests/GnuLdDriverTest.cpp?rev=243060&r1=243059&r2=243060&view=diff
==============================================================================
--- lld/trunk/unittests/DriverTests/GnuLdDriverTest.cpp (original)
+++ lld/trunk/unittests/DriverTests/GnuLdDriverTest.cpp Thu Jul 23 18:03:55 2015
@@ -42,7 +42,7 @@ protected:
     std::error_code ec =
         GnuLdDriver::evalLinkerScript(*_ctx, std::move(mb), out, nostdlib);
     EXPECT_FALSE(ec);
-  };
+  }
 
   std::unique_ptr<ELFLinkingContext> _ctx;
 };
@@ -281,4 +281,3 @@ TEST_F(LinkerScriptTest, ExprEval) {
   EXPECT_EQ(0x14000, result);
   EXPECT_EQ(0, sa2->symbol().compare(StringRef(".")));
 }
-