commit c53b3b080956305d9cdae9099da6722cf0844647
author Rui Ueyama <ruiu@google.com> Tue Dec 10 06:19:09 2013 +0000
committer Rui Ueyama <ruiu@google.com> Tue Dec 10 06:19:09 2013 +0000
tree 1945ed567ae7f5815865dbf647e7577ec05c882c
parent 7ce0fe015e61da103db42985a8f19537146ae920

Style fixes. No functionality change.

llvm-svn: 196883

lld/lib/Core/Resolver.cpp

diff --git a/lld/lib/Core/Resolver.cpp b/lld/lib/Core/Resolver.cpp
index f067568..1a86609 100644
--- a/lld/lib/Core/Resolver.cpp
+++ b/lld/lib/Core/Resolver.cpp
@@ -41,10 +41,9 @@
     if (_liveAtoms.count(atom))
       return false;
     // don't remove if marked never-dead-strip
-    if (const DefinedAtom *defAtom = dyn_cast<DefinedAtom>(atom)) {
+    if (const DefinedAtom *defAtom = dyn_cast<DefinedAtom>(atom))
       if (defAtom->deadStrip() == DefinedAtom::deadStripNever)
         return false;
-    }
     // do remove this atom
     return true;
   }
@@ -249,16 +248,14 @@
   _atoms.push_back(&atom);
 
   // tell symbol table
-  if (atom.scope() != Atom::scopeTranslationUnit) {
+  if (atom.scope() != Atom::scopeTranslationUnit)
     _symbolTable.add(atom);
-  }
 }
 
 // utility to add a vector of atoms
 void Resolver::addAtoms(const std::vector<const DefinedAtom *> &newAtoms) {
-  for (const DefinedAtom *newAtom : newAtoms) {
+  for (const DefinedAtom *newAtom : newAtoms)
     this->doDefinedAtom(*newAtom);
-  }
 }
 
 // Keep adding atoms until _context.nextFile() returns an error. This function
@@ -363,9 +360,8 @@
   }
 
   // mark all roots as live, and recursively all atoms they reference
-  for (const Atom *dsrAtom : _deadStripRoots) {
+  for (const Atom *dsrAtom : _deadStripRoots)
     this->markLive(*dsrAtom);
-  }
 
   // now remove all non-live atoms from _atoms
   _atoms.erase(
@@ -443,10 +439,9 @@
     return false;
   this->updateReferences();
   this->deadStripOptimize();
-  if (this->checkUndefines(false)) {
+  if (this->checkUndefines(false))
     if (!_context.allowRemainingUndefines())
       return false;
-  }
   this->removeCoalescedAwayAtoms();
   this->linkTimeOptimize();
   this->_result->addAtoms(_atoms);

lld/lib/Passes/GOTPass.cpp

diff --git a/lld/lib/Passes/GOTPass.cpp b/lld/lib/Passes/GOTPass.cpp
index f29c35b..9827952 100644
--- a/lld/lib/Passes/GOTPass.cpp
+++ b/lld/lib/Passes/GOTPass.cpp
@@ -100,8 +100,8 @@
   }
 
   // add all created GOT Atoms to master file
-  for (auto &it : targetToGOT) {
+  for (auto &it : targetToGOT)
     mergedFile->addAtom(*it.second);
-  }
 }
-}
+
+} // end namesapce lld

lld/lib/Passes/LayoutPass.cpp

diff --git a/lld/lib/Passes/LayoutPass.cpp b/lld/lib/Passes/LayoutPass.cpp
index 61b5393..272042b 100644
--- a/lld/lib/Passes/LayoutPass.cpp
+++ b/lld/lib/Passes/LayoutPass.cpp
@@ -416,19 +416,17 @@
         // Check if the current atom is part of the chain
         bool isAtomInChain = false;
         const DefinedAtom *lastAtom = rootAtom;
-        while (true) {
+        for (;;) {
           AtomToAtomT::iterator followOnAtomsIter =
-                  _followOnNexts.find(lastAtom);
+              _followOnNexts.find(lastAtom);
           if (followOnAtomsIter != _followOnNexts.end()) {
             lastAtom = followOnAtomsIter->second;
-            if (lastAtom == ai) {
-              isAtomInChain = true;
-              break;
-            }
+            if (lastAtom != ai)
+              continue;
+            isAtomInChain = true;
           }
-          else
-            break;
-        } // findAtomInChain
+          break;
+        }
 
         if (!isAtomInChain)
           _followOnNexts[lastAtom] = ai;
@@ -517,9 +515,8 @@
       for (const DefinedAtom *nextAtom = start->second; nextAtom != NULL;
            nextAtom = _followOnNexts[nextAtom]) {
         AtomToOrdinalT::iterator pos = _ordinalOverrideMap.find(nextAtom);
-        if (pos == _ordinalOverrideMap.end()) {
+        if (pos == _ordinalOverrideMap.end())
           _ordinalOverrideMap[nextAtom] = index++;
-        }
       }
     }
   }