Fix comments.

llvm-svn: 205568

lld/include/lld/ReaderWriter/ELFLinkingContext.h

@@ -88,7 +88,7 @@ public:
   }
   bool validateImpl(raw_ostream &diagnostics) override;
 
-  /// \brief Does the linker allow dynamic libraries to be linked with ?
+  /// \brief Does the linker allow dynamic libraries to be linked with?
   /// This is true when the output mode of the executable is set to be
   /// having NMAGIC/OMAGIC
   virtual bool allowLinkWithDynamicLibraries() const {
@@ -129,7 +129,7 @@ public:
   /// \brief Does the output have dynamic sections.
   virtual bool isDynamic() const;
 
-  /// \brief Are we creating a shared library ?
+  /// \brief Are we creating a shared library?
   virtual bool isDynamicLibrary() const {
     return _outputELFType == llvm::ELF::ET_DYN;
   }

lld/lib/Core/Resolver.cpp

@@ -178,7 +178,7 @@ void Resolver::doUndefinedAtom(const UndefinedAtom &atom) {
 
 /// \brief Add the section group and the group-child reference members.
 bool Resolver::maybeAddSectionGroupOrGnuLinkOnce(const DefinedAtom &atom) {
-  // First time adding a group ?
+  // First time adding a group?
   bool isFirstTime = _symbolTable.addGroup(atom);
 
   if (!isFirstTime) {

lld/lib/ReaderWriter/ELF/Chunk.h

@@ -73,7 +73,7 @@ public:
   // Memory size of the chunk
   uint64_t memSize() const { return _msize; }
   void setMemSize(uint64_t msize) { _msize = msize; }
-  // Whats the contentType of the chunk ?
+  // Whats the contentType of the chunk?
   virtual int getContentType() const = 0;
   // Writer the chunk
   virtual void write(ELFWriter *writer, TargetLayout<ELFT> &layout,

lld/lib/ReaderWriter/ELF/DefaultLayout.h

@@ -600,7 +600,7 @@ template <class ELFT> void DefaultLayout<ELFT>::assignSectionsToSegments() {
   ScopedTask task(getDefaultDomain(), "assignSectionsToSegments");
   ELFLinkingContext::OutputMagic outputMagic = _context.getOutputMagic();
     // TODO: Do we want to give a chance for the targetHandlers
-    // to sort segments in an arbitrary order ?
+    // to sort segments in an arbitrary order?
   // sort the sections by their order as defined by the layout
   std::stable_sort(_sections.begin(), _sections.end(),
                    [](Chunk<ELFT> *A, Chunk<ELFT> *B) {
@@ -702,7 +702,7 @@ template <class ELFT> void DefaultLayout<ELFT>::assignSectionsToSegments() {
 
 template <class ELFT> void DefaultLayout<ELFT>::assignFileOffsets() {
   // TODO: Do we want to give a chance for the targetHandlers
-  // to sort segments in an arbitrary order ?
+  // to sort segments in an arbitrary order?
   std::sort(_segments.begin(), _segments.end(), Segment<ELFT>::compareSegments);
   int ordinal = 0;
   // Compute the number of segments that might be needed, so that the

lld/lib/ReaderWriter/ELF/ELFFile.h

@@ -605,7 +605,7 @@ template <class ELFT> error_code ELFFile<ELFT>::createAtoms() {
   for (auto &i : _sectionSymbols) {
     const Elf_Shdr *section = i.first;
 
-    // Check if need to create atoms for this section ?
+    // Check if need to create atoms for this section?
     if ((ignoreCreateAtomsForSection(section)))
       continue;
 

lld/lib/ReaderWriter/ELF/SectionChunks.h

@@ -429,7 +429,7 @@ public:
     _virtualAddr = addr;
   }
 
-  // Is the section loadable ?
+  // Is the section loadable?
   inline bool isLoadableSection() const { return _isLoadableSection; }
 
   // Set section Loadable

lld/lib/ReaderWriter/FileArchive.cpp

@@ -67,7 +67,7 @@ public:
     return result[0].release();
   }
 
-  /// \brief Load all members of the archive ?
+  /// \brief Load all members of the archive?
   virtual bool isWholeArchive() const { return _isWholeArchive; }
 
   /// \brief parse each member

lld/unittests/DriverTests/InputGraphTest.cpp

@@ -40,7 +40,7 @@ class MyExpandFileNode : public SimpleFileNode {
 public:
   MyExpandFileNode(StringRef path) : SimpleFileNode(path) {}
 
-  /// \brief How do we want to expand the current node ?
+  /// \brief How do we want to expand the current node?
   bool shouldExpand() const override { return true; }
 
   /// \brief Get the elements that we want to expand with.